From ac5b2ee947297d9347db22d2a2ad873bbcaed356 Mon Sep 17 00:00:00 2001 From: buslov-artem Date: Wed, 25 Oct 2017 21:00:41 +0300 Subject: [PATCH] Fix V575 warning from PVS-Studio Static Analyzer The potential null pointer is passed into 'sprintf' function. --- src/acllib.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/acllib.c b/src/acllib.c index eaec288..fce7cb2 100644 --- a/src/acllib.c +++ b/src/acllib.c @@ -810,10 +810,12 @@ void loadSound(const char *fileName,ACL_Sound *pSound) int len = strlen(fileName)*sizeof(char); len +=64; cmdStr = (char*)malloc(len); - sprintf(cmdStr,"open \"%s\" type mpegvideo alias S%d",fileName,g_soundID); - *pSound = g_soundID; - ++g_soundID; - mciSendStringA(cmdStr,NULL,0,NULL); + if (cmdStr != NULL) { + sprintf(cmdStr,"open \"%s\" type mpegvideo alias S%d",fileName,g_soundID); + *pSound = g_soundID; + ++g_soundID; + mciSendStringA(cmdStr,NULL,0,NULL); + } free(cmdStr); }