search

ARMmbed / mbed-client-c

DEPRECATED: Coap+mbed-client-c libraries.
https://cloud.mbed.com/docs/current
Other
2 stars 11 forks source link

Valgrind unittest fixes #90

Closed TeroJaasko closed 8 years ago

TeroJaasko commented 8 years ago

Starting point: ./run_unit_tests_with_valgrind.sh 2>&1 >/dev/null |grep ERROR ==20487== ERROR SUMMARY: 10 errors from 10 contexts (suppressed: 0 from 0) ==20488== ERROR SUMMARY: 1976 errors from 16 contexts (suppressed: 0 from 0) ==20489== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==20490== ERROR SUMMARY: 297 errors from 66 contexts (suppressed: 0 from 0) ==20491== ERROR SUMMARY: 23 errors from 17 contexts (suppressed: 0 from 0) ==20492== ERROR SUMMARY: 4 errors from 4 contexts (suppressed: 0 from 0)

TeroJaasko commented 8 years ago

Current state (sn_coap_builder_unit_tests is still left): ./run_unit_tests_with_valgrind.sh 2>&1 >/dev/null |grep ERROR ==4571== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==4572== ERROR SUMMARY: 1976 errors from 16 contexts (suppressed: 0 from 0) ==4573== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==4574== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==4575== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==4576== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

TeroJaasko commented 8 years ago

Much better now: ./run_unit_tests_with_valgrind.sh 2>&1 >/dev/null |grep ERROR ==13090== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==13091== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==13092== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==13093== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==13094== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) ==13095== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

TeroJaasko commented 8 years ago

@yogpan01 @AnttiKauppila @anttiylitokola

yogpan01 commented 8 years ago

I will take this change in ASAP as it cleans our table with valgrind issues.

TeroJaasko commented 8 years ago

rebuild