Changeset 89cc0cc


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

More assertion cleanups

Location:
lib
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • lib/libtrace.h.in

    r33b735c r89cc0cc  
    324324        /** Trace err stat **/
    325325        TRACE_ERR_STAT = -20,
     326        /** Unable to create deadtrace **/
     327        TRACE_ERR_CREATE_DEADTRACE = -21,
    326328};
    327329
     
    15441546 *
    15451547 */
    1546 DLLEXPORT void trace_destroy(libtrace_t *trace);
     1548DLLEXPORT int trace_destroy(libtrace_t *trace);
    15471549
    15481550/** Close a dummy trace file, freeing up any resources it may have been using
    15491551 * @param trace         The dummy trace to be destroyed
    15501552 */
    1551 DLLEXPORT void trace_destroy_dead(libtrace_t *trace);
     1553DLLEXPORT int trace_destroy_dead(libtrace_t *trace);
    15521554
    15531555/** Close an output trace, freeing up any resources it may have been using
    15541556 * @param trace         The output trace to be destroyed
    15551557 */
    1556 DLLEXPORT void trace_destroy_output(libtrace_out_t *trace);
     1558DLLEXPORT int trace_destroy_output(libtrace_out_t *trace);
    15571559
    15581560/** Flush an output trace, forcing any buffered packets to be written
     
    17081710 */
    17091711DLLEXPORT
    1710 void trace_get_thread_statistics(libtrace_t *trace, libtrace_thread_t *t,
     1712int trace_get_thread_statistics(libtrace_t *trace, libtrace_thread_t *t,
    17111713                                 libtrace_stat_t *stats);
    17121714
     
    17421744 */
    17431745DLLEXPORT
    1744 void trace_subtract_statistics(const libtrace_stat_t *a,
     1746int trace_subtract_statistics(const libtrace_stat_t *a,
    17451747                               const libtrace_stat_t *b, libtrace_stat_t *c);
    17461748
     
    17541756 */
    17551757DLLEXPORT
    1756 void trace_add_statistics(const libtrace_stat_t *a,
     1758int trace_add_statistics(const libtrace_stat_t *a,
    17571759                          const libtrace_stat_t *b, libtrace_stat_t *c);
    17581760
     
    19421944 */
    19431945DLLEXPORT
    1944 void trace_construct_packet(libtrace_packet_t *packet,
     1946int trace_construct_packet(libtrace_packet_t *packet,
    19451947                libtrace_linktype_t linktype, const void *data, uint16_t len);
    19461948
  • 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.