Unconstrained download to a directory w/o permission crashes udm

Bug #1411866 reported by Barry Warsaw
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-download-manager (Ubuntu)
Fix Released
Critical
Manuel de la Peña

Bug Description

One of my system-image test suites creates a destination directory that is unwritable. Previously, when UDM tried to download a file to this while unconstrained, it would issue an error with "Permission denied" in the error output. Now however, udm appears to either crash or freeze. It becomes unresponsive to D-Bus and the error signal is never received, thus failing the test.

Further, since it usually freezes, this affects future tests because udm will not exit when its D-Bus exit method is called. Eventually it gets signal killed and then the tests can proceed.

For now, I will have to skip these tests when s-i runs under udm. The same tests pass for the curl-based downloader.

% dpkg-query -W ubuntu-download-manager
ubuntu-download-manager 0.9+15.04.20141202-0ubuntu1

Related branches

Revision history for this message
Barry Warsaw (barry) wrote :

I added some debugging; here's how to reproduce:

$ bzr branch lp:system-image $ cd system-image $ tox -e py34-udm --notest -r $
SYSTEMIMAGE_PYCURL=0 SYSTEMIMAGE_DBUS_DAEMON_HUP_SLEEP_SECONDS=2
SYSTEMIMAGE_REACTOR_TIMEOUT=60 .tox/py34-udm/bin/python -m nose2 -vv -P
TestAPI.test_state_machine_exceptions

I am watching the D-Bus signals and all I see is the `started` signal. After
that, nothing and the dbus loop times out. It doesn't matter how long I crank
the timeout to, no other signals are ever received.

Changed in ubuntu-download-manager (Ubuntu):
assignee: nobody → Manuel de la Peña (mandel)
importance: Undecided → Critical
status: New → Triaged
Revision history for this message
Barry Warsaw (barry) wrote :

Ping?

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-download-manager - 1.3+16.10.20160909.1-0ubuntu1

---------------
ubuntu-download-manager (1.3+16.10.20160909.1-0ubuntu1) yakkety; urgency=medium

  [ Florian Boucault ]
  * Fixes for crossbuilding.

  [ Michael Sheldon ]
  * Make user visible strings translatable (LP: #1618892)
  * Fix queue blocking and error reporting when file path is unwritable
    (LP: #1411866)
  * Remove dependency on qt5-default metapackage

 -- Michael Sheldon <email address hidden> Fri, 09 Sep 2016 11:49:03 +0000

Changed in ubuntu-download-manager (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.