Handle arrays too large to fit in iceConn buffers

Fixes numerous gcc warnings of the form:

sm_client.c: In function ‘SmcOpenConnection’:
SMlibint.h:109:25: warning: potential null pointer dereference [-Wnull-dereference]
     *((CARD32 *) _pBuf) = _val; \
SMlibint.h:160:5: note: in expansion of macro ‘STORE_CARD32’
     STORE_CARD32 (_pBuf, (CARD32) _len); \
     ^~~~~~~~~~~~
sm_client.c:207:5: note: in expansion of macro ‘STORE_ARRAY8’
     STORE_ARRAY8 (pData, len, previousId);
     ^~~~~~~~~~~~

v2: Raise required libICE version to 1.1.0 to get the updated
IceGetHeaderExtra macro definition needed for this to work correctly.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

1 files changed, 47 insertions, 16 deletions

diff --git a/src/sm_client.c b/src/sm_client.c
index 3c939b8..4552819 100644
--- a/src/sm_client.c
+++ b/src/sm_client.c
@@ -204,9 +204,13 @@ SmcOpenConnection(char *networkIdsList, SmPointer context,
 	SIZEOF (smRegisterClientMsg), WORD64COUNT (extra),
 	smRegisterClientMsg, pMsg, pData);
 
-    STORE_ARRAY8 (pData, len, previousId);
-
-    IceFlush (iceConn);
+    if (pData != NULL) {
+	STORE_ARRAY8 (pData, len, previousId);
+	IceFlush (iceConn);
+    }
+    else {
+	SEND_ARRAY8 (iceConn, len, previousId);
+    }
 
     replyWait.sequence_of_request = IceLastSentSequenceNumber (iceConn);
     replyWait.major_opcode_of_request = _SmcOpcode;
@@ -260,9 +264,13 @@ SmcOpenConnection(char *networkIdsList, SmPointer context,
 		    SIZEOF (smRegisterClientMsg), WORD64COUNT (extra),
 		    smRegisterClientMsg, pMsg, pData);
 
-		STORE_ARRAY8 (pData, 0, "");
-
-		IceFlush (iceConn);
+		if (pData != NULL) {
+		    STORE_ARRAY8 (pData, 0, "");
+		    IceFlush (iceConn);
+		}
+		else {
+		    SEND_ARRAY8 (iceConn, 0, "");
+		}
 
 		replyWait.sequence_of_request =
 		    IceLastSentSequenceNumber (iceConn);
@@ -296,13 +304,24 @@ SmcCloseConnection(SmcConn smcConn, int count, char **reasonMsgs)
 	SIZEOF (smCloseConnectionMsg), WORD64COUNT (extra),
 	smCloseConnectionMsg, pMsg, pData);
 
-    STORE_CARD32 (pData, (CARD32) count);
-    pData += 4;
+    if (pData != NULL) {
+	STORE_CARD32 (pData, (CARD32) count);
+	STORE_CARD32 (pData, (CARD32) 0); /* padding */
 
-    for (i = 0; i < count; i++)
-	STORE_ARRAY8 (pData, strlen (reasonMsgs[i]), reasonMsgs[i]);
+	for (i = 0; i < count; i++)
+	    STORE_ARRAY8 (pData, strlen (reasonMsgs[i]), reasonMsgs[i]);
 
-    IceFlush (iceConn);
+	IceFlush (iceConn);
+    } else {
+	CARD32 count_header[2] = {
+	    (CARD32) count,
+	    (CARD32) 0 /* padding */
+	};
+	IceWriteData32 (iceConn, 8, count_header);
+
+	for (i = 0; i < count; i++)
+	    SEND_ARRAY8 (iceConn, strlen (reasonMsgs[i]), reasonMsgs[i]);
+    }
 
     IceProtocolShutdown (iceConn, _SmcOpcode);
     IceSetShutdownNegotiation (iceConn, False);
@@ -412,13 +431,25 @@ SmcDeleteProperties(SmcConn smcConn, int numProps, char **propNames)
 	SIZEOF (smDeletePropertiesMsg), WORD64COUNT (extra),
 	smDeletePropertiesMsg, pMsg, pData);
 
-    STORE_CARD32 (pData, numProps);
-    pData += 4;
+    if (pData != NULL) {
+	STORE_CARD32 (pData, (CARD32) numProps);
+	STORE_CARD32 (pData, (CARD32) 0); /* padding */
 
-    for (i = 0; i < numProps; i++)
-	STORE_ARRAY8 (pData, strlen (propNames[i]), propNames[i]);
+	for (i = 0; i < numProps; i++)
+	    STORE_ARRAY8 (pData, strlen (propNames[i]), propNames[i]);
 
-    IceFlush (iceConn);
+	IceFlush (iceConn);
+    }
+    else {
+	CARD32 count_header[2] = {
+	    (CARD32) numProps,
+	    (CARD32) 0 /* padding */
+	};
+	IceWriteData32 (iceConn, 8, count_header);
+
+	for (i = 0; i < numProps; i++)
+	    SEND_ARRAY8 (iceConn, strlen (propNames[i]), propNames[i]);
+    }
 }