Re: [Tcpreplay-users] Tcpreplay Randomizing IP address question

[Aaron T. <syn...@gm...>]

Thanks Dileep.  In the future it would be easier for me if you sent a
patch using diff, but this is small enough where I can apply it by
hand.

On Tue, Jun 28, 2011 at 8:07 AM, Dileep Simha <ds...@gm...> wrote:
> Hey
> I've been working on fixing the bug with the Timing and the CTRL-C
> calculation and i THINK i fixed it.  I edited four files.  The change I made
> in tcpreplay,c doesn;t have anything to do with the timer, I just added a
> line of code to re-randomize ip addresses after each loop iteration.  I have
> done some testing with this code and I believe it does work.  However, I
> don't know if my changes could cause the code to break in different areas.
>  I believe the only place where break_now is called is in send_packets.
>  With my tests, when I send a sigint to tcpreplay on a loop, it ends and
> gives what I believe is a correct output.  I've tested the output with
> tcpdump to make sure it is correct.  I just wanted to know if someone might
> see a potential problem that could arise with these changes.  I've attached
> the files I have changed and put the output of diff with the original file.
> Thanks,
> Dileep
>
> TCPREPLAY.C
> 160a161,166
>>
>> //////////////////////////////////DILEEP////////////////////////////////////////////////
>>             #ifdef TCPREPLAY_EDIT
>>               OPT_VALUE_SEED += options.loop;
>>               tcpedit_post_args(&tcpedit);
>>                       #endif
>>
>> //////////////////////////////////DILEEP////////////////////////////////////////////////
>
>
> SENDPACKETS.C
> 121c121
> <             break_now(0,options);
> ---
>>             break_now(0);
>
> ABORT.C
> 35a36
>> #include "../send_packets.h"
> 38a40
>> #include <sys/time.h>
> 66c68
> < break_now(int signo)
> ---
>> break_now(int signo, tcpreplay_opt_t options)
> 78a81,83
>>
>>         gettimeofday(&end, NULL);
>>
> 80c85,92
> <         exit(1);
> ---
>>
>>        // if (options != NULL){
>>                       printf("%s", sendpacket_getstat(options.intf1));
>>                       if (options.intf2 != NULL)
>>                               printf("%s",
>> sendpacket_getstat(options.intf2));
>>        // }
>>       exit(1);
>>
> ABORT.H
> 37c37
> < void break_now(int signo);
> ---
>> void break_now(int signo, tcpreplay_opt_t options)
> ------------------------------------------------------------------------------
> All of the data generated in your IT infrastructure is seriously valuable.
> Why? It contains a definitive record of application performance, security
> threats, fraudulent activity, and more. Splunk takes this data and makes
> sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-d2d-c2
> _______________________________________________
> Tcpreplay-users mailing list
> Tcp...@li...
> https://lists.sourceforge.net/lists/listinfo/tcpreplay-users
> Support Information: http://tcpreplay.synfin.net/trac/wiki/Support
>



-- 
Aaron Turner
http://synfin.net/         Twitter: @synfinatic
http://tcpreplay.synfin.net/ - Pcap editing and replay tools for Unix & Windows
Those who would give up essential Liberty, to purchase a little temporary
Safety, deserve neither Liberty nor Safety.
    -- Benjamin Franklin
"carpe diem quam minimum credula postero"