ScoopDroid/android_system_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

am 51a2e4d5: Merge "libsysutils: fix null pointer and memory leak issue"

Browse source 
* commit '51a2e4d5d44a6f788da207dff301356e16b4e283':
  libsysutils: fix null pointer and memory leak issue
This commit is contained in:
Colin Cross 2013-05-21 14:39:19 -07:00 • committed by Android Git Automerger
commit a9e08d3ed6
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
libsysutils/src/SocketClient.cpp
View file

@ -112,6 +112,12 @@ char *SocketClient::quoteArg(const char *arg) {
char *result = (char *)malloc(len * 2 + 3);
char *current = result;
const char *end = arg + len;
char *oldresult;
if(result == NULL) {
SLOGW("malloc error (%s)", strerror(errno));
return NULL;
}
*(current++) = '"';
while (arg < end) {
@ -125,8 +131,9 @@ char *SocketClient::quoteArg(const char *arg) {
}
*(current++) = '"';
*(current++) = '\0';
oldresult = result; // save pointer in case realloc fails
result = (char *)realloc(result, current-result);
return result;
return result ? result : oldresult;
}