mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-08-28 11:31:16 +00:00
glibcompat: Reimplement g_strdup_printf() and g_strdup_vprintf()
These functions don't really abort() on OOM. The fix was merged upstream, but not in the minimal version we require. Provide our own implementation which can be removed once we bump the minimal version. Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
parent
b2c2a3ae91
commit
9706476254
@ -64,6 +64,7 @@
|
|||||||
#include "libvirt/virterror.h"
|
#include "libvirt/virterror.h"
|
||||||
|
|
||||||
#include "c-strcase.h"
|
#include "c-strcase.h"
|
||||||
|
#include "glibcompat.h"
|
||||||
|
|
||||||
/* Merely casting to (void) is not sufficient since the
|
/* Merely casting to (void) is not sufficient since the
|
||||||
* introduction of the "warn_unused_result" attribute
|
* introduction of the "warn_unused_result" attribute
|
||||||
|
@ -1485,6 +1485,11 @@ virSecurityManagerTransactionStart;
|
|||||||
virSecurityManagerVerify;
|
virSecurityManagerVerify;
|
||||||
|
|
||||||
|
|
||||||
|
# util/glibcompat.h
|
||||||
|
vir_g_strdup_printf;
|
||||||
|
vir_g_strdup_vprintf;
|
||||||
|
|
||||||
|
|
||||||
# util/viralloc.h
|
# util/viralloc.h
|
||||||
virAlloc;
|
virAlloc;
|
||||||
virAllocN;
|
virAllocN;
|
||||||
|
@ -3,6 +3,8 @@
|
|||||||
# These files are not related to driver APIs. Simply generic
|
# These files are not related to driver APIs. Simply generic
|
||||||
# helper APIs for various purposes
|
# helper APIs for various purposes
|
||||||
UTIL_SOURCES = \
|
UTIL_SOURCES = \
|
||||||
|
util/glibcompat.c \
|
||||||
|
util/glibcompat.h \
|
||||||
util/viralloc.c \
|
util/viralloc.c \
|
||||||
util/viralloc.h \
|
util/viralloc.h \
|
||||||
util/virarch.c \
|
util/virarch.c \
|
||||||
|
51
src/util/glibcompat.c
Normal file
51
src/util/glibcompat.c
Normal file
@ -0,0 +1,51 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2019 Red Hat, Inc.
|
||||||
|
*
|
||||||
|
* This library is free software; you can redistribute it and/or
|
||||||
|
* modify it under the terms of the GNU Lesser General Public
|
||||||
|
* License as published by the Free Software Foundation; either
|
||||||
|
* version 2.1 of the License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This library is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
||||||
|
* Lesser General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Lesser General Public
|
||||||
|
* License along with this library. If not, see
|
||||||
|
* <http://www.gnu.org/licenses/>.
|
||||||
|
*/
|
||||||
|
|
||||||
|
#include <config.h>
|
||||||
|
|
||||||
|
#include "glibcompat.h"
|
||||||
|
|
||||||
|
#undef g_strdup_printf
|
||||||
|
#undef g_strdup_vprintf
|
||||||
|
|
||||||
|
/* Due to a bug in glib, g_strdup_printf() nor g_strdup_vprintf()
|
||||||
|
* abort on OOM. It's fixed in glib's upstream. Provide our own
|
||||||
|
* implementation until the fix gets distributed. */
|
||||||
|
char *
|
||||||
|
vir_g_strdup_printf(const char *msg, ...)
|
||||||
|
{
|
||||||
|
va_list args;
|
||||||
|
char *ret;
|
||||||
|
va_start(args, msg);
|
||||||
|
ret = g_strdup_vprintf(msg, args);
|
||||||
|
if (!ret)
|
||||||
|
abort();
|
||||||
|
va_end(args);
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
char *
|
||||||
|
vir_g_strdup_vprintf(const char *msg, va_list args)
|
||||||
|
{
|
||||||
|
char *ret;
|
||||||
|
ret = g_strdup_vprintf(msg, args);
|
||||||
|
if (!ret)
|
||||||
|
abort();
|
||||||
|
return ret;
|
||||||
|
}
|
31
src/util/glibcompat.h
Normal file
31
src/util/glibcompat.h
Normal file
@ -0,0 +1,31 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2019 Red Hat, Inc.
|
||||||
|
*
|
||||||
|
* This library is free software; you can redistribute it and/or
|
||||||
|
* modify it under the terms of the GNU Lesser General Public
|
||||||
|
* License as published by the Free Software Foundation; either
|
||||||
|
* version 2.1 of the License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This library is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
||||||
|
* Lesser General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Lesser General Public
|
||||||
|
* License along with this library. If not, see
|
||||||
|
* <http://www.gnu.org/licenses/>.
|
||||||
|
*/
|
||||||
|
|
||||||
|
#pragma once
|
||||||
|
|
||||||
|
#include <glib.h>
|
||||||
|
|
||||||
|
char *vir_g_strdup_printf(const char *msg, ...)
|
||||||
|
G_GNUC_PRINTF(1, 2);
|
||||||
|
char *vir_g_strdup_vprintf(const char *msg, va_list args)
|
||||||
|
G_GNUC_PRINTF(1, 0);
|
||||||
|
|
||||||
|
#if !GLIB_CHECK_VERSION(2, 64, 0)
|
||||||
|
# define g_strdup_printf vir_g_strdup_printf
|
||||||
|
# define g_strdup_vprintf vir_g_strdup_vprintf
|
||||||
|
#endif
|
Loading…
Reference in New Issue
Block a user