Attachment 81613 Details for Bug 66300 – [PATCH 2/4] Remove invoke of va_end before va_start

[patch] [PATCH 2/4] Remove invoke of va_end before va_start

0002-Remove-invoke-of-va_end-before-va_start.patch (text/plain), 1.08 KB, created by Chengwei Yang on 2013-06-28 06:53:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chengwei Yang

Created: 2013-06-28 06:53:05 UTC

Size: 1.08 KB

patch

obsolete

>From 93e8df27a1e4791789ad6c4ef69d6495476ce673 Mon Sep 17 00:00:00 2001
>From: Chengwei Yang <chengwei.yang@intel.com>
>Date: Fri, 28 Jun 2013 14:34:22 +0800
>Subject: [PATCH 2/4] Remove invoke of va_end before va_start
>
>Signed-off-by: Chengwei Yang <chengwei.yang@intel.com>
>---
> dbus/dbus-errors.c |    3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/dbus/dbus-errors.c b/dbus/dbus-errors.c
>index db3305b..a0571a5 100644
>--- a/dbus/dbus-errors.c
>+++ b/dbus/dbus-errors.c
>@@ -235,7 +235,7 @@ dbus_error_free (DBusError *error)
>  * must ensure the name and message are global data that won't be
>  * freed. You probably want dbus_set_error() instead, in most cases.
>  * 
>- * @param error the error.or #NULL
>+ * @param error the error or #NULL
>  * @param name the error name (not copied!!!)
>  * @param message the error message (not copied!!!)
>  */
>@@ -379,7 +379,6 @@ dbus_set_error (DBusError  *error,
>                                 message_from_error (name)))
>         {
>           _dbus_string_free (&str);
>-          va_end (args);
>           goto nomem;
>         }
>     }
>-- 
>1.7.9.5
>

Actions: View | Splinter Review

Attachments on bug 66300: 81612 | 81613 | 81615 | 81616

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.