]> Pileus Git - ~andy/linux/commit
projects / ~andy / linux / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 99d3d58) | patch
isdn: eicon: free pointer after using it in log msg in divas_um_idi_delete_entity()
authorJesper Juhl <jj@chaosbits.net>
Mon, 30 Sep 2013 19:25:27 +0000 (21:25 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 2 Oct 2013 20:00:59 +0000 (16:00 -0400)
commit84557783c5574f436826d88cd1bd035f20a4427f
treeac6c424c9a8c19864904737aa6d6f63326f03fdbtree | snapshot
parent99d3d587b2b4314ccc8ea066cb327dfb523d598ecommit | diff
isdn: eicon: free pointer after using it in log msg in divas_um_idi_delete_entity()

Not really a problem, but nice IMHO; the Coverity static analyzer
complains that we use the pointer 'e' after it has been freed, so move
the freeing below the final use, even if that use is just using the
value of the pointer and not actually dereferencing it.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/hardware/eicon/um_idi.c diff | blob | history
Andy's Linux Kernel repo