Attachment 42204 Details for Bug 33277 – [1/3] test_server_setup: on OOM while setting up watch/timeout, don't leak sd

[patch] [1/3] test_server_setup: on OOM while setting up watch/timeout, don't leak sd

0001-test_server_setup-on-OOM-while-setting-up-watch-time.patch (text/plain), 1.04 KB, created by Simon McVittie on 2011-01-19 09:42:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2011-01-19 09:42:50 UTC

Size: 1.04 KB

patch

obsolete

>From dae7a0c5eba6d05a2c595c86bcf85cc7839f9b40 Mon Sep 17 00:00:00 2001
>From: Simon McVittie <simon.mcvittie@collabora.co.uk>
>Date: Wed, 19 Jan 2011 17:28:58 +0000
>Subject: [PATCH 1/3] test_server_setup: on OOM while setting up watch/timeout, don't leak sd
>
>Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
>---
> test/test-utils.c |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/test/test-utils.c b/test/test-utils.c
>index eda20cd..67e207c 100644
>--- a/test/test-utils.c
>+++ b/test/test-utils.c
>@@ -304,7 +304,7 @@ test_server_setup (DBusLoop      *loop,
>                                         sd,
>                                         serverdata_free))
>     {
>-      return FALSE;
>+      goto nomem;
>     }
> 
>   if (!dbus_server_set_timeout_functions (server,
>@@ -313,7 +313,7 @@ test_server_setup (DBusLoop      *loop,
>                                           NULL,
>                                           sd, serverdata_free))
>     {
>-      return FALSE;
>+      goto nomem;
>     }   
>   return TRUE;
> 
>-- 
>1.7.2.3
>

Actions: View | Splinter Review

Attachments on bug 33277: 42204 | 42205 | 42206 | 42225 | 42269

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.