From bc068034babd222099c74612e7ab6c91e9a565fe Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Tue, 9 Sep 2014 17:09:58 +0200 Subject: [PATCH] util: process: Don't report OOM errors in helper virProcessTranslateStatus is used on error paths that should not spoil the returned error. As the errors are ignored, use the quiet versions of virAsprintf to create the message. --- src/util/virprocess.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/util/virprocess.c b/src/util/virprocess.c index 97cce4fc5f..5bb2298241 100644 --- a/src/util/virprocess.c +++ b/src/util/virprocess.c @@ -73,13 +73,13 @@ virProcessTranslateStatus(int status) { char *buf; if (WIFEXITED(status)) { - ignore_value(virAsprintf(&buf, _("exit status %d"), - WEXITSTATUS(status))); + ignore_value(virAsprintfQuiet(&buf, _("exit status %d"), + WEXITSTATUS(status))); } else if (WIFSIGNALED(status)) { - ignore_value(virAsprintf(&buf, _("fatal signal %d"), - WTERMSIG(status))); + ignore_value(virAsprintfQuiet(&buf, _("fatal signal %d"), + WTERMSIG(status))); } else { - ignore_value(virAsprintf(&buf, _("invalid value %d"), status)); + ignore_value(virAsprintfQuiet(&buf, _("invalid value %d"), status)); } return buf; }