system/freebsd-src
system/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src synced 2024-07-21 18:27:22 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Make sure that each va_start has one and only one matching va_end,

Browse source 
especially in error cases.
This commit is contained in:
Kevin Lo 2012-09-28 07:51:30 +00:00
parent e4b8a2fc5a
commit ba670ce023
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=241021
6 changed files with 17 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c
View file

@ -678,6 +678,7 @@ dt_printf(dtrace_hdl_t *dtp, FILE *fp, const char *format, ...)
dtp->dt_buffered_offs += needed;
assert(dtp->dt_buffered_buf[dtp->dt_buffered_offs] == '\0');
va_end(ap);
return (0);
}

2
cddl/contrib/opensolaris/lib/libzfs/common/libzfs_util.c
View file

@ -368,6 +368,7 @@ zfs_standard_error_fmt(libzfs_handle_t *hdl, int error, const char *fmt, ...)
case ENOSPC:
case EDQUOT:
zfs_verror(hdl, EZFS_NOSPC, fmt, ap);
va_end(ap);
return (-1);
case EEXIST:
@ -467,6 +468,7 @@ zpool_standard_error_fmt(libzfs_handle_t *hdl, int error, const char *fmt, ...)
case ENOSPC:
case EDQUOT:
zfs_verror(hdl, EZFS_NOSPC, fmt, ap);
va_end(ap);
return (-1);
case EAGAIN:

4
contrib/telnet/telnetd/state.c
View file

@ -1600,8 +1600,10 @@ output_data(const char *format, ...)
char *buf;
va_start(args, format);
if ((len = vasprintf(&buf, format, args)) == -1)
if ((len = vasprintf(&buf, format, args)) == -1) {
va_end(args);
return -1;
}
output_datalen(buf, len);
va_end(args);
free(buf);

12
usr.bin/csup/proto.c
View file

@ -789,18 +789,24 @@ proto_printf(struct stream *wr, const char *format, ...)
case '%':
n = stream_write(wr, "%", 1);
if (n == -1)
if (n == -1) {
va_end(ap);
return (-1);
}
break;
}
if (rv == -1)
if (rv == -1) {
va_end(ap);
return (-1);
}
fmt = cp + 1;
}
if (*fmt != '\0') {
rv = stream_printf(wr, "%s", fmt);
if (rv == -1)
if (rv == -1) {
va_end(ap);
return (-1);
}
}
done:
va_end(ap);

1
usr.sbin/pkg_install/lib/exec.c
View file

@ -93,6 +93,7 @@ vpipe(const char *fmt, ...)
fp = popen(cmd, "r");
if (fp == NULL) {
warnx("popen() failed");
va_end(args);
return NULL;
}
get_string(rp, MAXPATHLEN, fp);

1
usr.sbin/pw/pwupd.c
View file

@ -102,6 +102,7 @@ pwdb(char *arg,...)
if (WEXITSTATUS(i))
i = EIO;
}
va_end(ap);
return i;
}