commit e376f9339897df86b63f3c827f4750b7ce7d10c0
author Dave Wallace <dwallacelf@gmail.com> Sun Nov 19 11:20:02 2017 -0500
committer Dave Wallace <dwallacelf@gmail.com> Sun Nov 19 11:20:02 2017 -0500
tree 4278c86ebfd6781e809c8c396f5ebd54d002f676
parent cabbee7d8caeafa3959f4559177ae335567aefb4

VCL-LDPRELOAD: Fix CID179207

Change-Id: Ia6b9e39d5a2ef064cf71bfe87cb4b4a0648d735d
Signed-off-by: Dave Wallace <dwallacelf@gmail.com>

src/vcl/vcom_socket.c

diff --git a/src/vcl/vcom_socket.c b/src/vcl/vcom_socket.c
index 6450edd..e87fd8a 100644
--- a/src/vcl/vcom_socket.c
+++ b/src/vcl/vcom_socket.c
@@ -1557,8 +1557,17 @@
 
   do
     {
-      bytes_to_read = vppcom_session_attr (out_sid,
-					   VPPCOM_ATTR_GET_NWRITE, 0, 0);
+      rv = vppcom_session_attr (out_sid, VPPCOM_ATTR_GET_NWRITE, 0, 0);
+      if (rv < 0)
+	{
+	  clib_warning ("[%d] ERROR: vppcom_session_attr (out_sid (%u), "
+			"VPPCOM_ATTR_GET_NWRITE, 0, 0) returned %d (%s)!",
+			getpid (), out_sid, rv, vppcom_retval_str (rv));
+	  vec_reset_length (vsm->io_buffer);
+	  return rv;
+	}
+
+      bytes_to_read = (size_t) rv;
       if (VCOM_DEBUG > 2)
 	clib_warning ("[%d] results %ld, n_bytes_left %lu, "
 		      "bytes_to_read %lu", getpid (), results,
@@ -1606,8 +1615,10 @@
       if (rv < 0)
 	{
 	  clib_warning ("[%d] ERROR: vppcom_session_write ("
-			"out_sid %u, io_buffer %p, nbytes %d) returned %d",
-			getpid (), out_sid, vsm->io_buffer, nbytes, rv);
+			"out_sid %u, io_buffer %p, nbytes %d) "
+			"returned %d (%s)",
+			getpid (), out_sid, vsm->io_buffer, nbytes,
+			rv, vppcom_retval_str (rv));
 	  if (results == 0)
 	    {
 	      vec_reset_length (vsm->io_buffer);

src/vcl/vppcom.h

diff --git a/src/vcl/vppcom.h b/src/vcl/vppcom.h
index 93b7173..45ff116 100644
--- a/src/vcl/vppcom.h
+++ b/src/vcl/vppcom.h
@@ -103,6 +103,10 @@
       st = "VPPCOM_EAGAIN";
       break;
 
+    case VPPCOM_ENOMEM:
+      st = "VPPCOM_ENOMEM";
+      break;
+
     case VPPCOM_EINVAL:
       st = "VPPCOM_EINVAL";
       break;