gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #4269 from jianqingdu/unstable

Browse Source 
fix not call va_end() when syncWrite() failed
This commit is contained in:
Salvatore Sanfilippo 2018-01-24 10:55:25 +01:00 committed by GitHub
commit 4aa2ecd98b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/replication.c
View File

@ -1330,7 +1330,8 @@ char *sendSynchronousCommand(int flags, int fd, ...) {
cmd = sdscat(cmd,arg); cmd = sdscat(cmd,arg);
} }
cmd = sdscatlen(cmd,"\r\n",2); cmd = sdscatlen(cmd,"\r\n",2);
va_end(ap);
/* Transfer command to the server. */ /* Transfer command to the server. */
if (syncWrite(fd,cmd,sdslen(cmd),server.repl_syncio_timeout*1000) if (syncWrite(fd,cmd,sdslen(cmd),server.repl_syncio_timeout*1000)
== -1) == -1)
@ -1340,7 +1341,6 @@ char *sendSynchronousCommand(int flags, int fd, ...) {
strerror(errno)); strerror(errno));
} }
sdsfree(cmd); sdsfree(cmd);
va_end(ap);
} }
/* Read the reply from the server. */ /* Read the reply from the server. */