Changeset 6920 in vbox for trunk/src/libs

Timestamp:

Feb 12, 2008 1:16:28 PM (17 years ago)

Author:

vboxsync

Message:

XPCOM/IPC/DConnect: For zero-sized arrays, we use the size of 1 because whether malloc(0) returns a null pointer or not (which is used in isNull()) is implementation-dependent according to the C standard.

File:

• trunk/src/libs/xpcom18a4/ipc/ipcd/extensions/dconnect/src/ipcDConnectService.cpp (modified) (1 diff)

trunk/src/libs/xpcom18a4/ipc/ipcd/extensions/dconnect/src/ipcDConnectService.cpp

@@ -1053 +1053 @@
       return rv;
 
-  void *arr = nsMemory::Alloc(size * elemSize);
+  // Note: for zero-sized arrays, we use the size of 1 because whether
+  // malloc(0) returns a null pointer or not (which is used in isNull())
+  // is implementation-dependent according to the C standard
+  void *arr = nsMemory::Alloc((size ? size : 1) * elemSize);
   if (arr == nsnull)
     return NS_ERROR_OUT_OF_MEMORY;