Changeset a0e5b65 for lib


Ignore:
Timestamp:
02/05/19 13:48:07 (20 months ago)
Author:
Jacob Van Walraven <jcv9@…>
Branches:
develop
Children:
7b4f5e2
Parents:
e0f2dca
Message:

NULL terminate all strings returned from the meta API

File:
1 edited

Legend:

Unmodified
Added
Removed

  • lib/format_pktmeta.c

    re0f2dca ra0e5b65  
    165165        }
    166166        /* Ensure the supplied memory allocation is enough, if not only fill
    167          * what we can */
     167         * what we can. */
    168168        if (spacelen > r->items[index].len) {
    169169                memcpy(space, r->items[index].data, r->items[index].len);
     170                space[r->items[index].len] = '\0';
    170171        } else {
    171172                memcpy(space, r->items[index].data, spacelen);
     173                space[spacelen] = '\0';
    172174        }
    173175        trace_destroy_meta(r);
    … …  
    219221        if (r->items[index].len > spacelen) {
    220222                memcpy(space, r->items[index].data, spacelen);
     223                space[spacelen] = '\0';
    221224        } else {
    222225                memcpy(space, r->items[index].data, r->items[index].len);
     226                space[r->items[index].len] = '\0';
    223227        }
    224228        trace_destroy_meta(r);
    … …  
    332336        char *addrstr = inet_ntoa(*(struct in_addr *)&addr);
    333337        memcpy(space, addrstr, spacelen);
     338        space[spacelen] = '\0';
    334339        return space;
    335340}
    … …  
    461466        if (r->items[index].len > spacelen) {
    462467                memcpy(space, r->items[index].data, spacelen);
     468                space[spacelen] = '\0';
    463469        } else {
    464470                memcpy(space, r->items[index].data, r->items[index].len);
     471                space[r->items[index].len] = '\0';
    465472        }
    466473        trace_destroy_meta(r);
    … …  
    503510        if (r->items[0].len > spacelen) {
    504511                memcpy(space, r->items[0].data, spacelen);
     512                space[spacelen] = '\0';
    505513        } else {
    506514                memcpy(space, r->items[0].data, r->items[0].len);
     515                space[r->items[0].len] = '\0';
    507516        }
    508517        trace_destroy_meta(r);
    … …  
    596605        if (r->items[index].len > spacelen) {
    597606                memcpy(space, r->items[index].data, spacelen);
     607                space[spacelen] = '\0';
    598608        } else {
    599609                memcpy(space, r->items[index].data, r->items[index].len);
     610                space[r->items[index].len] = '\0';
    600611        }
    601612        trace_destroy_meta(r);
    … …  
    637648        if (r->items[0].len > spacelen) {
    638649                memcpy(space, r->items[0].data, spacelen);
     650                space[spacelen] = '\0';
    639651        } else {
    640652                memcpy(space, r->items[0].data, r->items[0].len);
     653                space[r->items[0].len] = '\0';
    641654        }
    642655        trace_destroy_meta(r);
    … …  
    692705        if (r->items[0].len > spacelen) {
    693706                memcpy(space, r->items[0].data, spacelen);
     707                space[spacelen] = '\0';
    694708        } else {
    695709                memcpy(space, r->items[0].data, r->items[0].len);
     710                space[r->items[0].len] = '\0';
    696711        }
    697712        trace_destroy_meta(r);
    … …  
    713728        if (r->items[0].len > spacelen) {
    714729                memcpy(space, r->items[0].data, spacelen);
     730                space[spacelen] = '\0';
    715731        } else {
    716732                memcpy(space, r->items[0].data, r->items[0].len);
     733                space[r->items[0].len] = '\0';
    717734        }
    718735        trace_destroy_meta(r);
    … …  
    734751        if (r->items[0].len > spacelen) {
    735752                memcpy(space, r->items[0].data, spacelen);
     753                space[spacelen] = '\0';
    736754        } else {
    737755                memcpy(space, r->items[0].data, r->items[0].len);
    738         }
    739         trace_destroy_meta(r);
    740         return space;
    741 }
    742 
     756                space[r->items[0].len] = '\0';
     757        }
     758        trace_destroy_meta(r);
     759        return space;
     760}
     761
Note: See TracChangeset for help on using the changeset viewer.