diff options
| author |   Steven <sluong@cisco.com> | 2017-11-15 16:00:38 -0800 |
|---|---|---|
| committer | Steven <sluong@cisco.com> | 2017-11-15 16:07:36 -0800 |
| commit | 9d72120a6986e642c5b16c8650ad266188adf142 (patch) | |
| tree | 8f71966bdaa8651e911fbfa0ff8fefdf30cec7c4 | |
| parent | 93149bb4cd0a137b2a6da4b0d9f8cb2032e44734 (diff) | |
memif: fix uninitialized pointer read coverity error
Set the content of tmp.sock prior to calling memif_msg_send_disconnect.
Also fix the problem socket was not close in the same spot due to
error encountered.
Change-Id: I8f54ebad2250d1944afcc52e71d2a59da05362af
Signed-off-by: Steven <sluong@cisco.com>
| -rw-r--r-- | src/plugins/memif/socket.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/plugins/memif/socket.c b/src/plugins/memif/socket.c index 23ea24902e3..9aa4a2e2e90 100644 --- a/src/plugins/memif/socket.c +++ b/src/plugins/memif/socket.c @@ -289,7 +289,9 @@ memif_msg_receive_init (memif_if_t ** mifp, memif_msg_t * msg, return 0; error: + tmp.sock = sock; memif_msg_send_disconnect (&tmp, err); + memif_socket_close (&sock); return err; } |