Discussion:
[Bug 1790671] [NEW] /usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy
Launchpad Bug Tracker
2018-10-08 08:58:34 UTC
Permalink
You have been subscribed to a public bug by Rik Mills (rikmills):

[Impact]
pkcon what-provides and other tools querying codecs do not work, they crash (unless you happen to be lucky with your apt cache). Also, cancalled transactions crash as well, even if you're lucky with your apt cache, as the "matcher" object is deleted twice.

There are two reasons: A duplicate delete statement in providesCodec
entered when cancelling the lookup, and a invalid pointer dereference in
there.

[Test case]
The daemon should not crash as below, but should print a useful message.

$ lxc launch -e ubuntu:bionic bbb
$ lxc exec bbb apt update
$ lxc exec bbb -- apt -y install packagekit
$ lxc exec bbb pkcon what-provides "gstreamer1.0(decoder-audio/ac3)"
[...] The daemon crashed mid-transaction!

(empty lxd container seems to be able to reproduce easily)

This only tests the pointer dereference, I don't have a test for the
duplicate. But the code is obviously correct:

for ... [
if (m_cancel) {
// here used to be "delete matcher" - that's deleted
break;
}
}
delete matcher;

That is, matcher is always deleted once now, and was always deleted
twice when cancelling before.

[Regression potential]
I don't think it's possible to have a regression here, given the nature of the fix, but if there were one, we'd see different behavior in codec lookup.

For the duplicate delete on cancelled transactions, you'd be looking at
memory leaks if there were a regression.

[Other info]
The Ubuntu Error Tracker has been receiving reports about a problem regarding packagekit. This problem was most recently seen with package version 1.1.10-1ubuntu2, the problem page at https://errors.ubuntu.com/problem/46649a8a55e07e74b9d522c9bc9d71a74905ccc2 contains more details, including versions of packages affected, stacktrace or traceback, and individual crash reports.
If you do not have access to the Ubuntu Error Tracker and are a software developer, you can request it at http://forms.canonical.com/reports/.

** Affects: packagekit (Ubuntu)
Importance: High
Assignee: Julian Andres Klode (juliank)
Status: Fix Released

** Affects: packagekit (Ubuntu Bionic)
Importance: Undecided
Status: Fix Committed


** Tags: artful bionic cosmic kylin-17.10 verification-done verification-done-bionic
--
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy
https://bugs.launchpad.net/bugs/1790671
You received this bug notification because you are a member of Kubuntu Bugs, which is subscribed to the bug report.
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https://lists.ubuntu.com/mai
Julian Andres Klode
2018-10-11 14:30:30 UTC
Permalink
Putting this on hold a bit; once we have a fix for the other PackageKit
bug in the SRU queue, so we don't cause breakage again.

** Tags added: block-proposed
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https://
Brian Murray
2018-10-17 22:37:59 UTC
Permalink
Hello errors.ubuntu.com, or anyone else affected,

Accepted packagekit into bionic-proposed. The package will build now and
be available at
https://launchpad.net/ubuntu/+source/packagekit/1.1.9-1ubuntu2.18.04.3
in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed. Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested and change the tag from
verification-needed-bionic to verification-done-bionic. If it does not
fix the bug for you, please add a comment stating that, and change the
tag to verification-failed-bionic. In either case, without details of
your testing we will not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Tags removed: verification-done verification-done-bionic
** Tags added: verification-needed verification-needed-bionic
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https
Julian Andres Klode
2018-10-18 13:33:18 UTC
Permalink
Well, timing broke the test - it no longer fails in the unfixed version.
ugh.

** Tags removed: block-proposed
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
Julian Andres Klode
2018-10-18 13:39:30 UTC
Permalink
But well, the verification of 2.18.04.2 is still valid, the change to
the postinst does not invalidate it, so let's ignore that I'm not lucky
enough to end up with a broken apt cache.

** Tags removed: verification-needed verification-needed-bionic
** Tags added: verification-done verification-done-bionic
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https://lists.ubuntu.com/mailma
Łukasz Zemczak
2018-10-25 14:49:01 UTC
Permalink
The verification of the Stable Release Update for packagekit has
completed successfully and the package has now been released to
-updates. Subsequently, the Ubuntu Stable Release Updates Team is being
unsubscribed and will not receive messages about this bug report. In
the event that you encounter a regression using the package from
-updates please report a new bug using ubuntu-bug and tag the bug report
regression-update so we can easily find any regressions.
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/
Launchpad Bug Tracker
2018-10-25 14:49:12 UTC
Permalink
This bug was fixed in the package packagekit - 1.1.9-1ubuntu2.18.04.3

---------------
packagekit (1.1.9-1ubuntu2.18.04.3) bionic; urgency=medium

* Pass --no-restart-after-upgrade to dh_installsystemd to avoid PackageKit
restarting while upgrading under PackageKit (LP: #1790613)

packagekit (1.1.9-1ubuntu2.18.04.2) bionic; urgency=medium

* debian/patches/frontend-locking.diff:
Implement frontend locking in a simple way. Will need some more
work to upstream, and possibly some error checking. (LP: #1795614)
* Bump libapt-pkg-dev build-dep to >= 1.6.5~ for frontend locking
* debian/patches/aptcc-Fix-invalid-version-dereference-in-AptInf-prov.patch,
aptcc-removing-duplicate-delete-call.patch:
Fix invalid dereference and delete wrong (duplicate) "delete"
statement in providesCodec (LP: #1790671)

-- Julian Andres Klode <***@ubuntu.com> Mon, 15 Oct 2018 15:46:47
+0200

** Changed in: packagekit (Ubuntu Bionic)
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1790671

Title:
/usr/lib/packagekit/packagekitd:11:std::__cxx11::basic_string:AptIntf::providesCodec:backend_what_provides_thread:pk_backend_job_thread_setup:g_thread_proxy

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/packagekit/+bug/1790671/+subscriptions
--
kubuntu-bugs mailing list
kubuntu-***@lists.ubuntu.com
https://lists.ubunt
Loading...