Fix: RESOURCE_LEAK (CWE-404)
authorTomas Hozza <thozza@redhat.com>
Sun, 24 Feb 2013 10:38:15 +0000 (11:38 +0100)
committerHendrik Sattler <post@hendrik-sattler.de>
Sun, 24 Feb 2013 10:38:15 +0000 (11:38 +0100)
commitc61a4cc585bf66b359ff655889fb3a833477ff68
tree3f954e9a19fd68a2beb0cfe7509fb59f93bd1c59
parentc23f2351cae3177527f826a2d9f593dbe09cd1f7
Fix: RESOURCE_LEAK (CWE-404)

Coverity output:
apps/lib/obex_put_common.c:59: cond_true: Condition
 "OBEX_ObjectGetNextHeader(handle, object, &hi, &hv, &hlen)", taking true branch
apps/lib/obex_put_common.c:60: switch: Switch case value "1"
apps/lib/obex_put_common.c:65: switch_case: Reached case "1"
apps/lib/obex_put_common.c:66: alloc_fn: Storage is returned from allocation
 function "malloc(size_t)".
apps/lib/obex_put_common.c:66: var_assign: Assigning: "namebuf" = storage
 returned from "malloc(hlen / 2U)".
apps/lib/obex_put_common.c:66: cond_true: Condition
 "namebuf = malloc(hlen / 2)", taking true branch
apps/lib/obex_put_common.c:67: noescape: Resource "(uint8_t *)namebuf" is not
 freed or pointed-to in function
 "OBEX_UnicodeToChar(uint8_t *, uint8_t const *, int)"
lib/lib/obex.c:899:41: noescape:
 "OBEX_UnicodeToChar(uint8_t *, uint8_t const *, int)" does not free or save
  its pointer parameter "c".
apps/lib/obex_put_common.c:68: var_assign: Assigning: "name" = "namebuf".
apps/lib/obex_put_common.c:70: break: Breaking from switch
apps/lib/obex_put_common.c:82: switch_end: Reached end of switch
apps/lib/obex_put_common.c:83: loop: Jumping back to the beginning of the loop
apps/lib/obex_put_common.c:59: loop_begin: Jumped back to beginning of loop
apps/lib/obex_put_common.c:59: cond_false: Condition
 "OBEX_ObjectGetNextHeader(handle, object, &hi, &hv, &hlen)", taking false
  branch
apps/lib/obex_put_common.c:83: loop_end: Reached end of loop
apps/lib/obex_put_common.c:84: cond_true: Condition "!body", taking true branch
apps/lib/obex_put_common.c:86: leaked_storage: Variable "namebuf" going out of
 scope leaks the storage it points to.
apps/lib/obex_put_common.c:86: leaked_storage: Variable "name" going out of
 scope leaks the storage it points to.
apps/lib/obex_put_common.c