diff options
| author | David Lamparter <equinox@diac24.net> | 2019-06-12 22:39:18 +0200 | 
|---|---|---|
| committer | David Lamparter <equinox@diac24.net> | 2019-06-13 13:35:33 +0200 | 
| commit | bb53f1bc6664c3ae3cac8a74014d708624490e65 (patch) | |
| tree | 2c0019696e4e12ca90ea367d0cae78115a417c72 /ldpd/packet.c | |
| parent | 3239e3ca3b407fa355c3a1b6bfb8a9bbe9843923 (diff) | |
ldpd: fix clang-SA warning in packet reading
clang-SA thinks we can skip the SA but have something allocated in buf.
Signed-off-by: David Lamparter <equinox@diac24.net>
Diffstat (limited to 'ldpd/packet.c')
| -rw-r--r-- | ldpd/packet.c | 5 | 
1 files changed, 5 insertions, 0 deletions
diff --git a/ldpd/packet.c b/ldpd/packet.c index 8ca90841de..dfab30eeb3 100644 --- a/ldpd/packet.c +++ b/ldpd/packet.c @@ -618,12 +618,17 @@ session_read(struct thread *thread)  			len -= msg_size;  		}  		free(buf); +		buf = NULL;  		if (len != 0) {  			session_shutdown(nbr, S_BAD_PDU_LEN, 0, 0);  			return (0);  		}  	} +	/* shouldn't happen, session_get_pdu should be > 0 if buf was +	 * allocated - but let's get rid of the SA warning. +	 */ +	free(buf);  	return (0);  }  | 
