proxmox-mirrors/qemu
1
0
Fork 0
mirror of https://git.proxmox.com/git/qemu synced 2025-08-07 20:08:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Do not abort on qemu_malloc(0) in production builds

Browse Source 
qemu_malloc() does not allow size=0 to be passed in and aborts on this behavior.

Unfortunately, there is good reason to believe that within qemu, there are a
number of, so far, undetected places that assume size=0 can be safely passed.
Since we do not want to abort unnecessarily in production builds, return
qemu_malloc(1) whenever the version file indicates that this is a production
build.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Anthony Liguori 2009-12-09 12:59:36 -06:00
parent 725b8a6983
commit 20ff6c8066
2 changed files with 30 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
configure vendored
View File

@ -256,6 +256,7 @@ blobs="yes"
pkgversion="" pkgversion=""
check_utests="no" check_utests="no"
user_pie="no" user_pie="no"
zero_malloc=""
# OS specific # OS specific
if check_define __linux__ ; then if check_define __linux__ ; then
@ -1792,8 +1793,9 @@ fi
# Consult white-list to determine whether to enable werror # Consult white-list to determine whether to enable werror
# by default. Only enable by default for git builds # by default. Only enable by default for git builds
z_version=`cut -f3 -d. $source_path/VERSION`
if test -z "$werror" ; then if test -z "$werror" ; then
z_version=`cut -f3 -d. $source_path/VERSION`
if test "$z_version" = "50" -a \ if test "$z_version" = "50" -a \
"$linux" = "yes" ; then "$linux" = "yes" ; then
werror="yes" werror="yes"
@ -1802,6 +1804,16 @@ if test -z "$werror" ; then
fi fi
fi fi
# Disable zero malloc errors for official releases unless explicitly told to
# enable/disable
if test -z "$zero_malloc" ; then
if test "$z_version" = "50" ; then
zero_malloc="no"
else
zero_malloc="yes"
fi
fi
if test "$werror" = "yes" ; then if test "$werror" = "yes" ; then
QEMU_CFLAGS="-Werror $QEMU_CFLAGS" QEMU_CFLAGS="-Werror $QEMU_CFLAGS"
fi fi
@ -2109,6 +2121,10 @@ fi
echo "CONFIG_UNAME_RELEASE=\"$uname_release\"" >> $config_host_mak echo "CONFIG_UNAME_RELEASE=\"$uname_release\"" >> $config_host_mak
if test "$zero_malloc" = "yes" ; then
echo "CONFIG_ZERO_MALLOC=y" >> $config_host_mak
fi
# USB host support # USB host support
case "$usb" in case "$usb" in
linux) linux)

19
qemu-malloc.c
View File

@ -42,22 +42,29 @@ void qemu_free(void *ptr)
free(ptr); free(ptr);
} }
static int allow_zero_malloc(void)
{
#if defined(CONFIG_ZERO_MALLOC)
return 1;
#else
return 0;
#endif
}
void *qemu_malloc(size_t size) void *qemu_malloc(size_t size)
{ {
if (!size) { if (!size && !allow_zero_malloc()) {
abort(); abort();
} }
return oom_check(malloc(size)); return oom_check(malloc(size ? size : 1));
} }
void *qemu_realloc(void *ptr, size_t size) void *qemu_realloc(void *ptr, size_t size)
{ {
if (size) { if (size) {
return oom_check(realloc(ptr, size)); return oom_check(realloc(ptr, size));
} else { } else if (allow_zero_malloc()) {
if (ptr) { return oom_check(realloc(ptr, size ? size : 1));
return realloc(ptr, size);
}
} }
abort(); abort();
} }