Changeset 89cc0cc for lib/trace.c


Ignore:
Timestamp:
11/22/18 17:10:50 (4 years ago)
Author:
Jacob Van Walraven <jcv9@…>
Branches:
develop
Children:
25a3255
Parents:
33b735c
Message:

More assertion cleanups

File:
1 edited

Legend:

Unmodified
Added
Removed
  • lib/trace.c

    r33b735c r89cc0cc  
    731731 *
    732732 */
    733 DLLEXPORT void trace_destroy(libtrace_t *libtrace) {
     733DLLEXPORT int trace_destroy(libtrace_t *libtrace) {
    734734        int i;
    735735
    736         assert(libtrace);
    737         //if(!libtrace) {
    738         //      return TRACE_ERR_NULL_TRACE;
    739         //}
     736        /*assert(libtrace);*/
     737        if(!libtrace) {
     738                return TRACE_ERR_NULL_TRACE;
     739        }
    740740
    741741        ASSERT_RET(pthread_mutex_destroy(&libtrace->libtrace_lock), == 0);
     
    758758                trace_fin_packet(libtrace->last_packet);
    759759        }
    760         assert(libtrace->last_packet == NULL);
    761         //if (libtrace->last_packet != NULL) {
    762         //      trace_set_err(libtrace, TRACE_ERR_PAUSE_FIN, "Unable to remove all data stored against trace in trace_destroy()");
    763         //      return -1;
    764         //}
     760        /*assert(libtrace->last_packet == NULL);*/
     761        if (libtrace->last_packet != NULL) {
     762                trace_set_err(libtrace, TRACE_ERR_PAUSE_FIN, "Unable to remove all data stored against trace in trace_destroy()");
     763                return -1;
     764        }
    765765
    766766        if (libtrace->format) {
     
    826826
    827827        free(libtrace);
    828 }
    829 
    830 
    831 DLLEXPORT void trace_destroy_dead(libtrace_t *libtrace) {
    832         assert(libtrace);
    833         //if(!libtrace) {
    834         //      return TRACE_ERR_NULL_TRACE;
    835         //}
     828
     829        return 0;
     830}
     831
     832
     833DLLEXPORT int trace_destroy_dead(libtrace_t *libtrace) {
     834        /*assert(libtrace);*/
     835        if(!libtrace) {
     836                return TRACE_ERR_NULL_TRACE;
     837        }
    836838
    837839        ASSERT_RET(pthread_mutex_destroy(&libtrace->libtrace_lock), == 0);
     
    845847                free(libtrace->format_data);
    846848        free(libtrace);
     849
     850        return 0;
    847851}
    848852/* Close an output trace file, freeing up any resources it may have been using
     
    850854 * @param libtrace      the output trace file to be destroyed
    851855 */
    852 DLLEXPORT void trace_destroy_output(libtrace_out_t *libtrace) {
    853         assert(libtrace);
    854         //if(!libtrace) {
    855         //      return TRACE_ERR_NULL_TRACE;
    856         //}
     856DLLEXPORT int trace_destroy_output(libtrace_out_t *libtrace) {
     857        /*assert(libtrace);*/
     858        if(!libtrace) {
     859                return TRACE_ERR_NULL_TRACE;
     860        }
    857861        if (libtrace->format && libtrace->format->fin_output)
    858862                libtrace->format->fin_output(libtrace);
     
    860864                free(libtrace->uridata);
    861865        free(libtrace);
     866
     867        return 0;
    862868}
    863869
     
    22212227 */
    22222228DLLEXPORT
    2223 void trace_construct_packet(libtrace_packet_t *packet,
     2229int trace_construct_packet(libtrace_packet_t *packet,
    22242230                libtrace_linktype_t linktype,
    22252231                const void *data,
    2226                 uint16_t len)
    2227 {
     2232                uint16_t len) {
     2233
     2234        if (!packet) {
     2235                return TRACE_ERR_NULL_PACKET;
     2236        }
     2237
    22282238        size_t size;
    22292239        static libtrace_t *deadtrace=NULL;
     
    22552265
    22562266        /* Now fill in the libtrace packet itself */
    2257         assert(deadtrace);
     2267        /*assert(deadtrace);*/
     2268        if (!deadtrace) {
     2269                return TRACE_ERR_CREATE_DEADTRACE;
     2270        }
    22582271        packet->trace=deadtrace;
    22592272        size=len+sizeof(hdr);
     
    22782291
    22792292        trace_clear_cache(packet);
     2293
     2294        return 0;
    22802295}
    22812296
     
    24282443}
    24292444
    2430 void trace_get_thread_statistics(libtrace_t *trace, libtrace_thread_t *t,
     2445int trace_get_thread_statistics(libtrace_t *trace, libtrace_thread_t *t,
    24312446                                 libtrace_stat_t *stat)
    24322447{
    2433         assert(trace && stat);
    2434         assert(stat->magic == LIBTRACE_STAT_MAGIC && "Please use"
    2435                "trace_create_statistics() to allocate statistics");
     2448        /*assert(trace && stat);*/
     2449        if (!trace) {
     2450                return TRACE_ERR_NULL_TRACE;
     2451        }
     2452        if (!stat) {
     2453                trace_set_err(trace, TRACE_ERR_STAT, "Stat is NULL trace_get_thread_statistics()");
     2454                return -1;
     2455        }
     2456        /*assert(stat->magic == LIBTRACE_STAT_MAGIC && "Please use"
     2457               "trace_create_statistics() to allocate statistics");*/
     2458        if (!(stat->magic == LIBTRACE_STAT_MAGIC)) {
     2459                trace_set_err(trace, TRACE_ERR_STAT,
     2460                        "Use trace_create_statistics() to allocate statistics in trave_get_thread_statistics()");
     2461                return -1;
     2462        }
    24362463        stat->reserved1 = 0;
    24372464        stat->reserved2 = 0;
     
    24462473                trace->format->get_thread_statistics(trace, t, stat);
    24472474        }
    2448         return;
     2475
     2476        return 0;
    24492477}
    24502478
     
    24642492}
    24652493
    2466 void trace_subtract_statistics(const libtrace_stat_t *a, const libtrace_stat_t *b,
     2494int trace_subtract_statistics(const libtrace_stat_t *a, const libtrace_stat_t *b,
    24672495                         libtrace_stat_t *c) {
    2468         assert(a->magic == LIBTRACE_STAT_MAGIC && "Please use"
     2496
     2497        if (!(a->magic == LIBTRACE_STAT_MAGIC)
     2498                || !(b->magic == LIBTRACE_STAT_MAGIC)
     2499                || !(c->magic == LIBTRACE_STAT_MAGIC)) {
     2500                return TRACE_ERR_STAT;
     2501        }
     2502
     2503        /*assert(a->magic == LIBTRACE_STAT_MAGIC && "Please use"
    24692504               "trace_create_statistics() to allocate statistics");
    24702505        assert(b->magic == LIBTRACE_STAT_MAGIC && "Please use"
    24712506               "trace_create_statistics() to allocate statistics");
    24722507        assert(c->magic == LIBTRACE_STAT_MAGIC && "Please use"
    2473                "trace_create_statistics() to allocate statistics");
     2508               "trace_create_statistics() to allocate statistics");*/
    24742509
    24752510#define X(x) \
     
    24822517        LIBTRACE_STAT_FIELDS
    24832518#undef X
    2484 }
    2485 
    2486 void trace_add_statistics(const libtrace_stat_t *a, const libtrace_stat_t *b,
     2519        return 0;
     2520}
     2521
     2522int trace_add_statistics(const libtrace_stat_t *a, const libtrace_stat_t *b,
    24872523                         libtrace_stat_t *c) {
    2488         assert(a->magic == LIBTRACE_STAT_MAGIC && "Please use"
     2524        if (!(a->magic == LIBTRACE_STAT_MAGIC)
     2525                || !(b->magic == LIBTRACE_STAT_MAGIC)
     2526                || !(c->magic == LIBTRACE_STAT_MAGIC)) {
     2527                return TRACE_ERR_STAT;
     2528        }
     2529
     2530        /*assert(a->magic == LIBTRACE_STAT_MAGIC && "Please use"
    24892531               "trace_create_statistics() to allocate statistics");
    24902532        assert(b->magic == LIBTRACE_STAT_MAGIC && "Please use"
    24912533               "trace_create_statistics() to allocate statistics");
    24922534        assert(c->magic == LIBTRACE_STAT_MAGIC && "Please use"
    2493                "trace_create_statistics() to allocate statistics");
     2535               "trace_create_statistics() to allocate statistics");*/
    24942536
    24952537#define X(x) \
     
    25022544        LIBTRACE_STAT_FIELDS
    25032545#undef X
     2546        return 0;
    25042547}
    25052548
Note: See TracChangeset for help on using the changeset viewer.