conf_reload_path() calls nih_file_read(). Although the return value of
that call is checked, the code is wrong. It looks likely that this bug
crept in since, unfortunately, the documentation for nih_file_read() is
incorrect. It states:

    Returns: newly allocated string or NULL if insufficient memory.

... whereas it *should* state:

    Returns: newly allocated string or NULL on raised error.

As such, if the call to nih_file_read() fails, the error object should
be destroyed before continuing to avoid the crash seen above.

** Changed in: upstart (Ubuntu)
   Importance: Undecided => High

** Changed in: upstart (Ubuntu)
     Assignee: (unassigned) => James Hunt (jamesodhunt)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1430403

Title:
  ubuntu-touch livefs builds kill upstart in host

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/upstart/+bug/1430403/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to