silence use after the free clang analyzer warning
authorPetr Štetiar <ynezz@true.cz>
Tue, 29 Oct 2019 20:11:38 +0000 (21:11 +0100)
committerPetr Štetiar <ynezz@true.cz>
Tue, 29 Oct 2019 21:16:00 +0000 (22:16 +0100)
commit7ce813fcd667c80a936825e24a8090813bac75ed
treec462dcac790f1493d9867f630d8c58362221785f
parent1f73b6a8e67881834b0512d3e07aaaf17e71b49e
silence use after the free clang analyzer warning

scan-build from clang version 9 complains about following:

 nl.c:507:9: warning: Use of memory after it is freed
        while (nlmsg_ok(hdr, n)) {
               ^~~~~~~~~~~~~~~~

which seems to be impossible codepath as clang analyzer doesn't somehow
account properly nl_syserr2nlerr(errno) return value:

 } else {
    free(msg.msg_control);
    free(*buf);
    return -nl_syserr2nlerr(errno);
 }

which should be always < 0, but analyzer is still checking for > 0 code
path as well for some reason. So in order to make the analyzer happy,
set the buf pointer to NULL explicitly and add assert to make it clear,
that this codepath should never happen.

Signed-off-by: Petr Štetiar <ynezz@true.cz>
nl.c