[Bug 1871725] Re: python3-pykdl: PyKDL crashes Python 3 interpretter (SIGABRT) if any API accepting a str is used

[Marc Deslauriers]

** Also affects: orocos-kdl (Ubuntu Focal)
   Importance: Undecided
       Status: New

** Also affects: orocos-kdl (Ubuntu Groovy)
   Importance: Undecided
       Status: Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1871725

Title:
  python3-pykdl: PyKDL crashes Python 3 interpretter (SIGABRT) if any
  API accepting a str is used

Status in orocos-kdl package in Ubuntu:
  Fix Released
Status in orocos-kdl source package in Focal:
  New
Status in orocos-kdl source package in Groovy:
  Fix Released

Bug description:
  [Impact]

   * The package python3-pykdl in focal crashes the Python 3 interpreter
  if any API accepting a str is used.

   * python3-pykdl is used in the ROS robot control software
  (https://ros.org) based on Ubuntu.

   * The patch is taken from upstream and uses PyUnicode_AsUTF8, as
  provided by Python 3.

  [Test Case]

  $ python3 -c "import PyKDL; PyKDL.Tree('foobar')"
  python3: /build/orocos-kdl-oHbJfL/orocos-kdl-1.4.0/python_orocos_kdl/PyKDL/std_string.sip:52: int convertTo_std_string(PyObject*, void**, int*, PyObject*): Assertion `PyUnicode_Check(s)' failed.
  Aborted (core dumped)
  $ echo $?
  134

  [Regression Potential]

   * The patch is part of upstream git since 2018 [1], Debian bullseye
  [2] , Ubuntu groovy and has been accepted into Debian buster [3].

   * I don't see a regression potential as is basically the utf-8
  transition of Python 3.

  [1] https://github.com/orocos/orocos_kinematics_dynamics/commit/ee996a4442339010571cadd976fc6ed17c98dfab
  [2] https://tracker.debian.org/news/1117004/accepted-orocos-kdl-140-9-source-into-unstable/
  [3] https://tracker.debian.org/news/1119612/accepted-orocos-kdl-140-7deb10u1-source-into-proposed-updates-stable-new-proposed-updates/

  [Original Description]

  Dear Maintainer,

  The package python3-pykdl crashes the Python 3 interpreter if any API
  accepting a str is used. I've tested this using the latest Focal packages.

  Minimal reproducible example:

  $ python3 -c "import PyKDL; PyKDL.Tree('foobar')"
  python3: /build/orocos-kdl-mPkyII/orocos-kdl-1.4.0/python_orocos_kdl/PyKDL/std_string.sip:52: int convertTo_std_string(PyObject*, void**, int*, PyObject*): Assertion `PyUnicode_Check(s)' failed.
  Aborted (core dumped)
  $ echo $?
  134

  There is a patch fixing this issue in this upstream Debian Buster bug:
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=956254

  Would you be willing to apply that patch in Ubuntu Focal?

  Cheers,
  Shane

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/orocos-kdl/+bug/1871725/+subscriptions