Merge lp:~xnox/upstart/bug-1324096 into lp:~jamesodhunt/upstart/bug-1324096

Proposed by Dimitri John Ledkov
Status: Merged
Merged at revision: 1624
Proposed branch: lp:~xnox/upstart/bug-1324096
Merge into: lp:~jamesodhunt/upstart/bug-1324096
Diff against target: 27 lines (+10/-0)
1 file modified
init/control.c (+10/-0)
To merge this branch: bzr merge lp:~xnox/upstart/bug-1324096
Reviewer Review Type Date Requested Status
James Hunt Approve
Review via email: mp+222142@code.launchpad.net
To post a comment you must log in.
Revision history for this message
James Hunt (jamesodhunt) wrote :

Thanks - merged.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'init/control.c'
--- init/control.c 2014-06-04 15:32:58 +0000
+++ init/control.c 2014-06-05 09:25:56 +0000
@@ -1393,6 +1393,11 @@
1393 const char *var,1393 const char *var,
1394 int replace)1394 int replace)
1395{1395{
1396 if (! var) {
1397 nih_dbus_error_raise_printf (DBUS_ERROR_INVALID_ARGS,
1398 _("Variable may not be empty string"));
1399 return -1;
1400 }
1396 nih_local char **vars = NULL;1401 nih_local char **vars = NULL;
1397 1402
1398 vars = NIH_MUST (nih_str_array_new (NULL));1403 vars = NIH_MUST (nih_str_array_new (NULL));
@@ -1523,6 +1528,11 @@
1523 char * const *job_details,1528 char * const *job_details,
1524 const char *name)1529 const char *name)
1525{1530{
1531 if (! name) {
1532 nih_dbus_error_raise_printf (DBUS_ERROR_INVALID_ARGS,
1533 _("Variable may not be empty string"));
1534 return -1;
1535 }
1526 nih_local char **names = NULL;1536 nih_local char **names = NULL;
1527 1537
1528 names = NIH_MUST (nih_str_array_new (NULL));1538 names = NIH_MUST (nih_str_array_new (NULL));

Subscribers

People subscribed via source and target branches