Attachment 119578 Details for Bug 90021 – shell-test: Calling _dbus_string_init without checking return value (CID 60587).

[patch] shell-test: Calling _dbus_string_init without checking return value (CID 60587).

0001-shell-test-Calling-_dbus_string_init-without-checkin.patch (text/plain), 1.35 KB, created by Ralf Habacker on 2015-11-11 23:45:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ralf Habacker

Created: 2015-11-11 23:45:44 UTC

Size: 1.35 KB

patch

obsolete

>From fb7c185e1ad95e970f1bd1611c6df4597e61d803 Mon Sep 17 00:00:00 2001
>From: Ralf Habacker <ralf.habacker@freenet.de>
>Date: Tue, 10 Nov 2015 22:18:16 +0100
>Subject: [PATCH] shell-test: Calling _dbus_string_init without checking return
> value (CID 60587).
>
>Reported by Coverity: CID 60587: Unchecked return value (CHECKED_RETURN)
>
>Bug: https://bugs.freedesktop.org/show_bug.cgi?id=90021
>---
> test/shell-test.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
>diff --git a/test/shell-test.c b/test/shell-test.c
>index 61280d6..6fa7a88 100644
>--- a/test/shell-test.c
>+++ b/test/shell-test.c
>@@ -36,7 +36,13 @@ test_command_line_internal (dbus_bool_t should_work,
> 
>   original_argc = _dbus_list_get_length (&list);
>   original_argv = dbus_new (char *, original_argc);
>-  _dbus_string_init (&str);
>+  if (!_dbus_string_init (&str))
>+    {
>+      _dbus_list_clear (&list);
>+      dbus_free (original_argv);
>+      return FALSE;
>+    }
>+
>   for (i = 0, node = _dbus_list_get_first_link (&list); i < original_argc && node;
>        i++, node = _dbus_list_get_next_link (&list, node))
>     {
>@@ -57,6 +63,7 @@ test_command_line_internal (dbus_bool_t should_work,
>           should_work ? "" : " (as expected)",
>           error.message ? error.message : "");
>       dbus_free (original_argv);
>+      _dbus_string_free (&str);
>       return !should_work;
>     }
>   else
>-- 
>1.8.4.5
>

Actions: View | Splinter Review

Attachments on bug 90021: 115063 | 115064 | 115065 | 115068 | 115072 | 115073 | 115122 | 115234 | 115235 | 115236 | 115237 | 115246 | 115270 | 115576 | 115577 | 115578 | 115579 | 115580 | 115635 | 117015 | 119577 | 119578

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.