[Bug 1331555] Re: Trusty's libsepol1 causes issues with Precise's Upstart during dist-upgrade

[Michael Wisheu]

Dimitri, I've just encountered another unchecked 'telinit u' call
causing Upstart issues case. :-/

** Description changed:

  Trusty's libsepol1 (2.2-1) has no Upstart version check for the 'telinit
  u' call. During the dist-upgrade from Precise to Trusty this causes
  Upstart to loose its state. Because of this Upstart no longer knows
- which PID the dbus system bus process has and thus all service reloads
- fail.
+ which PIDs running services have and thus all service reloads fail.
  
  This bug is similarly to bug http://pad.lv/1313712 and the same fix
  applies.
  
  Faulty code:
  if [ "$1" = "configure" ]; then
   # Restart init. If it fails, there is nothing we can do, so
   # just ignore the error (NOTE: Borrowed from libc6.postinst)
   telinit u 2>/dev/null || true ; sleep 1
  fi
  
  Correct code:
  if [ "$1" = "configure" ]; then
   # Restart init. If it fails, there is nothing we can do, so
   # just ignore the error (NOTE: Borrowed from libc6.postinst)
   if dpkg --compare-versions "$UPSTART_VERSION_RUNNING" ge 1.6.1; then
    telinit u 2>/dev/null || true ; sleep 1
   fi
  fi

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

Title:
  Trusty's libsepol1 causes issues with Precise's Upstart during dist-
  upgrade

Status in “libsepol” package in Ubuntu:
  New

Bug description:
  Trusty's libsepol1 (2.2-1) has no Upstart version check for the
  'telinit u' call. During the dist-upgrade from Precise to Trusty this
  causes Upstart to loose its state. Because of this Upstart no longer
  knows which PIDs running services have and thus all service reloads
  fail.

  This bug is similarly to bug http://pad.lv/1313712 and the same fix
  applies.

  Faulty code:
  if [ "$1" = "configure" ]; then
   # Restart init. If it fails, there is nothing we can do, so
   # just ignore the error (NOTE: Borrowed from libc6.postinst)
   telinit u 2>/dev/null || true ; sleep 1
  fi

  Correct code:
  if [ "$1" = "configure" ]; then
   # Restart init. If it fails, there is nothing we can do, so
   # just ignore the error (NOTE: Borrowed from libc6.postinst)
   if dpkg --compare-versions "$UPSTART_VERSION_RUNNING" ge 1.6.1; then
    telinit u 2>/dev/null || true ; sleep 1
   fi
  fi

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