sal/osl/unx/pipe.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
New commits:
commit f6141d884b7a89a856a34dc93991dbe01ded0b6b
Author: José Guilherme Vanz <v...@libreoffice.org>
Date: Sat Nov 2 17:49:16 2013 -0200
fdo#71043 - Use STACK lint tool to clean code
The code is using a pointer without check if it is not null.
Change-Id: Icb2dcf8d41a35514e18a5881f59399951b3e0493
Reviewed-on: https://gerrit.libreoffice.org/6529
Tested-by: Caolán McNamara <caol...@redhat.com>
Reviewed-by: Caolán McNamara <caol...@redhat.com>
diff --git a/sal/osl/unx/pipe.c b/sal/osl/unx/pipe.c
index e271e94..79e847e 100644
--- a/sal/osl/unx/pipe.c
+++ b/sal/osl/unx/pipe.c
@@ -98,6 +98,8 @@ oslPipe __osl_createPipeImpl()
oslPipe pPipeImpl;
pPipeImpl = (oslPipe)calloc(1, sizeof(struct oslPipeImpl));
+ if (pPipeImpl == NULL)
+ return NULL;
pPipeImpl->m_nRefCount =1;
pPipeImpl->m_bClosed = sal_False;
#if defined(LINUX)
@@ -230,7 +232,13 @@ oslPipe SAL_CALL osl_psz_createPipe(const sal_Char
*pszPipeName, oslPipeOptions
}
/* alloc memory */
- pPipe= __osl_createPipeImpl();
+ pPipe = __osl_createPipeImpl();
+
+ if (pPipe == NULL)
+ {
+ OSL_TRACE("__osl_createPipe socket failed");
+ return NULL;
+ }
/* create socket */
pPipe->m_Socket = socket(AF_UNIX, SOCK_STREAM, 0);
@@ -467,7 +475,7 @@ oslPipe SAL_CALL osl_acceptPipe(oslPipe pPipe)
else
{
/* alloc memory */
- pAcceptedPipe= __osl_createPipeImpl();
+ pAcceptedPipe = __osl_createPipeImpl();
OSL_ASSERT(pAcceptedPipe);
if(pAcceptedPipe==NULL)
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
