Attachment 93553 Details for Bug 60847 – proposed patch

[patch] proposed patch

0001-PolkitAgentSession-fix-race-between-child-and-io-wat.patch (text/plain), 2.39 KB, created by Rui Tiago Matos on 2014-02-06 18:50:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rui Tiago Matos

Created: 2014-02-06 18:50:47 UTC

Size: 2.39 KB

patch

obsolete

>From 82a37c6e0305db75ba474231c79cebebc7ef9bdb Mon Sep 17 00:00:00 2001
>From: Rui Matos <tiagomatos@gmail.com>
>Date: Thu, 6 Feb 2014 18:41:18 +0100
>Subject: [PATCH] PolkitAgentSession: fix race between child and io watches
>
>The helper flushes and fdatasyncs stdout and stderr before terminating
>but this doesn't guarantee that our io watch is called before our
>child watch. This means that we can end up with a successful return
>from the helper which we still report as a failure.
>
>To fix this, instead of reporting a failure from the child watch
>handler, we'll just unset child_pid.
>
>In addition, we add G_IO_HUP and G_IO_ERR to the conditions we look
>for in the io watch so that if the child terminates abnormally we
>still run the io watch handler which will complete the session
>reporting a failure.
>---
> src/polkitagent/polkitagentsession.c | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
>diff --git a/src/polkitagent/polkitagentsession.c b/src/polkitagent/polkitagentsession.c
>index 1c7a2dc..a9d098b 100644
>--- a/src/polkitagent/polkitagentsession.c
>+++ b/src/polkitagent/polkitagentsession.c
>@@ -444,9 +444,7 @@ child_watch_func (GPid     pid,
>                WEXITSTATUS(status));
>     }
> 
>-  /* kill all the watches we have set up, except for the child since it has exited already */
>   session->child_pid = 0;
>-  complete_session (session, FALSE);
> }
> 
> static gboolean
>@@ -470,6 +468,12 @@ io_watch_have_data (GIOChannel    *channel,
>       goto out;
>     }
> 
>+  if (!(condition & G_IO_IN))
>+    {
>+      complete_session (session, FALSE);
>+      goto out;
>+    }
>+
>   g_io_channel_read_line (channel,
>                           &line,
>                           NULL,
>@@ -655,7 +659,8 @@ polkit_agent_session_initiate (PolkitAgentSession *session)
>   g_source_attach (session->child_watch_source, g_main_context_get_thread_default ());
> 
>   session->child_stdout_channel = g_io_channel_unix_new (session->child_stdout);
>-  session->child_stdout_watch_source = g_io_create_watch (session->child_stdout_channel, G_IO_IN);
>+  session->child_stdout_watch_source = g_io_create_watch (session->child_stdout_channel,
>+                                                          G_IO_IN | G_IO_ERR | G_IO_HUP);
>   g_source_set_callback (session->child_stdout_watch_source, (GSourceFunc) io_watch_have_data, session, NULL);
>   g_source_attach (session->child_stdout_watch_source, g_main_context_get_thread_default ());
> 
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 60847: 93553 | 93662 | 93677

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.