Changeset a7d1914


Ignore:
Timestamp:
09/27/06 15:53:27 (15 years ago)
Author:
Perry Lorier <perry@…>
Branches:
4.0.1-hotfixes, cachetimestamps, develop, dpdk-ndag, etsilive, getfragoff, help, libtrace4, master, ndag_format, pfring, rc-4.0.1, rc-4.0.2, rc-4.0.3, rc-4.0.4, ringdecrementfix, ringperformance, ringtimestampfixes
Children:
742a332
Parents:
41b7eab
Message:

Remove packet->size as noone really knew what it held anyway, and disagreements
were more easily settled by removing it than standardising one way or the other.

Files:
8 edited

Legend:

Unmodified
Added
Removed

  • examples/skeleton/event.c

    rbad6a93 ra7d1914  
    8888                                if (obj.size == -1)
    8989                                        return -1;
    90                                 return packet->size;
     90                                return 1;
    9191                               
    9292                        /* End of trace has been reached */

  • lib/format_duck.c

    r85a79b0 ra7d1914  
    194194        }
    195195
    196         packet->size = duck_size;
    197196        return numbytes;
    198197}
    … …  
    210209       
    211210        assert(OUTPUT->file);
    212         /* Common sense size checking */
    213         assert(packet->size < 400);
    214211
    215212        if (OUTPUT->dag_version == 0) {

  • lib/format_erf.c

    r5945854 ra7d1914  
    547547
    548548        packet->type = RT_DUCK_2_4;
    549         packet->size = sizeof(duck_inf);
    550549        if (!DUCK.dummy_duck)
    551550                DUCK.dummy_duck = trace_create_dead("duck:dummy");
    552551        packet->trace = DUCK.dummy_duck;
    553         return packet->size;
     552        return sizeof(duck_inf);
    554553}       
    555554#else
    … …  
    581580
    582581        packet->type = RT_DUCK_2_5;
    583         packet->size = sizeof(duckinf_t);
    584582        if (!DUCK.dummy_duck)
    585583                DUCK.dummy_duck = trace_create_dead("rt:localhost:3434");
    586584        packet->trace = DUCK.dummy_duck;       
    587         return packet->size;
     585        return sizeof(duck_inf);
    588586}       
    589587#endif
    … …  
    662660        DAG.diff -= size;
    663661
    664         if (packet->payload != NULL)
    665                 packet->size = trace_get_capture_length(packet) +
    666                         erf_get_framing_length(packet);
    667         else
    668                 packet->size = erf_get_framing_length(packet);
    669        
    670662        tv = trace_get_timeval(packet);
    671663        DUCK.last_pkt = tv.tv_sec;
    672664       
    673         return (packet->size);
     665        return packet->payload ? size : erf_get_framing_length(packet);
    674666}
    675667

  • lib/format_rt.c

    r01818c6 ra7d1914  
    515515        }
    516516        packet->type = RT_INFO->rt_hdr.type;
    517         packet->size = RT_INFO->rt_hdr.length;
    518517
    519518        if (packet->type >= RT_DATA_SIMPLE) {
    … …  
    580579        /* Return the number of bytes read from the stream */
    581580        RT_INFO->rt_hdr.type = RT_LAST;
    582         return packet->size;
     581        return RT_INFO->rt_hdr.length;
    583582}
    584583

  • lib/format_wag.c

    re0f9c80 ra7d1914  
    338338        packet->header = packet->buffer;
    339339        packet->payload=(char*)packet->buffer+trace_get_framing_length(packet);
    340         packet->size = trace_get_framing_length(packet) + trace_get_capture_length(packet);
    341340        return numbytes;
    342341}

  • lib/libtrace.h.in

    rd80422e ra7d1914  
    177177        void *header;           /**< pointer to the framing header */
    178178        void *payload;          /**< pointer to the link layer */
     179        void *buffer;           /**< allocated buffer */
     180        uint32_t type;          /**< rt protocol type for the packet */
    179181        buf_control_t buf_control; /**< who owns the memory */
    180         void *buffer;           /**< allocated buffer */
    181         size_t size;            /**< trace_get_framing_length()
    182                                  * +trace_get_capture_length() */
    183         uint32_t type;          /**< rt protocol type for the packet */
    184182} libtrace_packet_t;
    185183

  • lib/linktypes.c

    r8889370 ra7d1914  
    173173                packet->payload=tmpbuffer+trace_get_framing_length(packet);
    174174                packet->type=pcap_dlt_to_rt(TRACE_DLT_LINUX_SLL);
    175                 packet->size+=sizeof(libtrace_sll_header_t);
    176175                ((struct pcap_pkthdr*) packet->header)->caplen+=
    177176                        sizeof(libtrace_sll_header_t);

  • lib/trace.c

    r0453ca8 ra7d1914  
    632632        dest->payload=(void*)
    633633                ((char*)dest->buffer+trace_get_framing_length(packet));
    634         dest->size=packet->size;
    635634        dest->type=packet->type;
    636635        dest->buf_control=TRACE_CTRL_PACKET;
    … …  
    677676        if (libtrace->format->read_packet) {
    678677                do {
    679                         packet->size=libtrace->format->read_packet(libtrace,packet);
    680                         if (packet->size==(size_t)-1 || packet->size==0) {
    681                                 return packet->size;
     678                        size_t ret;
     679                        ret=libtrace->format->read_packet(libtrace,packet);
     680                        if (ret==(size_t)-1 || ret==0) {
     681                                return ret;
    682682                        }
    683683                        if (libtrace->filter) {
    … …  
    694694                                                libtrace->snaplen);
    695695                        }
    696                         return packet->size;
     696                        return ret;
    697697                } while(1);
    698698        }
    699699        trace_set_err(libtrace,TRACE_ERR_UNSUPPORTED,"This format does not support reading packets\n");
    700         packet->size=~0U;
    701700        return ~0U;
    702701}
    … …  
    714713        assert(packet);
    715714        /* Verify the packet is valid */
    716         assert(packet->size<65536);
    717         assert(packet->size>0);
    718715        assert(libtrace->started);
    719716
    … …  
    750747        double seconds = 0.0;
    751748        struct timeval ts;
    752 
    753         assert(packet->size>0 && packet->size<65536);
    754749
    755750        if (packet->trace->format->get_erf_timestamp) {
    … …  
    781776        uint64_t ts = 0;
    782777        double seconds = 0.0;
    783         assert(packet->size>0 && packet->size<65536);
    784778        if (packet->trace->format->get_timeval) {
    785779                /* timeval -> timeval */
    … …  
    820814        struct timeval tv;
    821815
    822         assert(packet->size>0 && packet->size<65536);
    823        
    824816        if (packet->trace->format->get_seconds) {
    825817                /* seconds->seconds */
    … …  
    838830}
    839831
    840 DLLEXPORT size_t trace_get_capture_length(const libtrace_packet_t *packet) {
    841 
    842         assert(packet->size<65536);
    843 
     832DLLEXPORT size_t trace_get_capture_length(const libtrace_packet_t *packet)
     833{
    844834        if (packet->trace->format->get_capture_length) {
    845835                return packet->trace->format->get_capture_length(packet);
    … …  
    858848 */
    859849DLLEXPORT size_t trace_get_wire_length(const libtrace_packet_t *packet){
    860         assert(packet->size>0 && packet->size<65536);
    861 
    862850        if (packet->trace->format->get_wire_length) {
    863851                return packet->trace->format->get_wire_length(packet);
    … …  
    10531041{
    10541042        assert(packet);
    1055         assert(packet->size>0 && packet->size<65536);
    10561043        if (packet->trace->format->set_direction) {
    10571044                return packet->trace->format->set_direction(packet,direction);
    … …  
    10721059{
    10731060        assert(packet);
    1074         assert(packet->size>0 && packet->size<65536);
    10751061        if (packet->trace->format->get_direction) {
    10761062                return packet->trace->format->get_direction(packet);
    … …  
    12071193DLLEXPORT size_t trace_set_capture_length(libtrace_packet_t *packet, size_t size) {
    12081194        assert(packet);
    1209         assert(packet->size>0 && packet->size<65536);
    12101195
    12111196        if (packet->trace->format->set_capture_length) {
    12121197                int caplen=packet->trace->format->set_capture_length(packet,size);
    12131198                if (caplen!=-1) {
    1214                         packet->size=trace_get_framing_length(packet)+caplen;
     1199                        trace_get_framing_length(packet)+caplen;
    12151200                }
    12161201                return caplen;
    … …  
    14231408                uint16_t len)
    14241409{
     1410        size_t size;
    14251411        libtrace_t *deadtrace=NULL;
    14261412        libtrace_pcapfile_pkt_hdr_t hdr;
    … …  
    14341420
    14351421        packet->trace=deadtrace;
    1436         packet->size=len+sizeof(hdr);
     1422        size=len+sizeof(hdr);
    14371423        if (packet->buf_control==TRACE_CTRL_PACKET) {
    1438                 packet->buffer=realloc(packet->buffer,packet->size);
     1424                packet->buffer=realloc(packet->buffer,size);
    14391425        }
    14401426        else {
    1441                 packet->buffer=malloc(packet->size);
     1427                packet->buffer=malloc(size);
    14421428        }
    14431429        packet->buf_control=TRACE_CTRL_PACKET;
Note: See TracChangeset for help on using the changeset viewer.