Not closing modems when disconnected

Bug #424603 reported by Noel J. Bergman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
modemmanager (Ubuntu)
Fix Released
Undecided
Noel J. Bergman
network-manager (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: network-manager

NM 0.8, including the code upstream and in PPA, does not properly (tell Modem Manager to) close the serial device for a modem when disconnecting due to user action or system error. Modem Manager will therefore keep it open until Modem Manager closes.

Amongst other things, in current Karmic, this results in ttyACM0, 1, 2, 3, ... being created when the previous name is still in use due to not being closed.

Bug is confirmed and work is in progress to address it.

Noel J. Bergman (noeljb)
Changed in network-manager (Ubuntu):
status: New → Confirmed
Revision history for this message
Noel J. Bergman (noeljb) wrote :

A preliminary patch has been committed upstream, and should be in the PPA tomorrow.

Changed in network-manager (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Noel J. Bergman (noeljb) wrote :

The part of this bug that related to hot-unplugging devices while still in use was fixed in modemmanager. It related to the kernel not generating udev remove notices for tty devices when the parent usb device of a tty device was removed, so modemmanager will now look for the usb removal of the parent as well.

Upstream commit 53af144f49b0d81bd4dc1f5ee9eea6d61ccae992

Changed in modemmanager (Ubuntu):
assignee: nobody → Noel J. Bergman (noeljb)
status: New → Fix Committed
Revision history for this message
Noel J. Bergman (noeljb) wrote :

Commit for network manager fix bee58e359d16c867341b33d3293011310312386c

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

This bug was fixed in the package network-manager - 0.8~a~git.20090911t130220.4c77fa0-0ubuntu1

---------------
network-manager (0.8~a~git.20090911t130220.4c77fa0-0ubuntu1) karmic; urgency=low

  * upstream snapshot 2009-09-11 13:02:20 (GMT)
    + 4c77fa08623f41dd710c545ec859dd7e942a3f2c
    + fix LP: #411668 - NetworkManager crashes in g_type_check_instance_cast ()
      when connecting to 3G
    + fix LP: #366772 - Network manager does not show EVDO, device detected by hal
    + fix LP: #424092 - ifupdown connection device locking regressions caused by
      system-settings rework
    + fix LP: #424603 - Not closing modems when disconnected

  [ Tony Espy <email address hidden> ]
  * fix build error due to upstream re-factoring
    - update debian/patches/lp291564_ifupdown_unmanage_mapping_and_iface.patch
  * fix install problem due to new PolicyKit changes
    - update debian/network-manager.install
  * fix install problem due to libnm-glib naming changes
    - update debian/libnm-glib-dev.install
    - update debian/libnm-glib0.install
  * handle upstream soname version bump for libnm-glib
    - update debian/control
    - rename debian/libnm-glib0.install => debian/libnm-glib2.install
    - rename debian/libnm-glib0.symbols => debian/libnm-glib2.symbols
    - update debian/libnm-glib2.symbols
  * add new olpc_mesg symbols to libnm-util1.symbols
    - update debian/libnm-utils1.symbols
  * fix lintian errors: update Standards-Version to 3.8.3, add missing
    ${misc:Depends}, and replace deprecated ${Source-Version} variables
    with ${binary:Version}
    - update debian/control

  [ Alexander Sack <email address hidden> ]
  * drop -0ubuntu1 suffix from ip6 symbols added in previous upload
    - update debian/libnm-glib0.symbols
  * remove hal from build-depends and depends (LP: #416988)
    - update debian/control
  * adjust patches to new upstream codebase
    - update debian/patches/lp291564_ifupdown_unmanage_mapping_and_iface.patch
    - update debian/patches/lp291902_IFUPDOWN_dont_export_connection_in_unmanaged_mode.patch
  * build depend on libpolkit-gobject-1-dev instead of libpolkit-dbus-dev
    - update debian/control
  * make GET_SOURCE not go mad for git versions that use a different abbrev-id syntax
    - update debian/rules
  * drop patches applied upstream
    - delete debian/patches/lp291564_ifupdown_unmanage_mapping_and_iface.patch
    - delete debian/patches/lp291902_IFUPDOWN_dont_export_connection_in_unmanaged_mode.patch
    - update debian/patches/series

  [ Mathieu Trudel <email address hidden> ]
  * adjust patches to new upstream codebase
    - update debian/patches/lp291564_ifupdown_unmanage_mapping_and_iface.patch
  * add rule to retrieve current snapshot information
    - update debian/rules
  * fix missing character in GET_SOURCE code
    - update debian/rules
  * use current snapshot hash from changelog instead of latest in get-snapshot-info
    - update debian/rules
  * update GET_SOURCE code: exit on any error
    - update debian/rules

 -- Alexander Sack <email address hidden> Mon, 14 Sep 2009 11:29:38 +0200

Changed in network-manager (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package modemmanager - 0.2.git.20090909t142854.64b7be7-0ubuntu1

---------------
modemmanager (0.2.git.20090909t142854.64b7be7-0ubuntu1) karmic; urgency=low

  New upstream snapshot 2009-09-09 14:28:54 (GMT)
  + commit: 64b7be7460c3a9dbb3c9981de5754b330a8d2cdd
  + fix LP: #414604 - Modem Manager setting bad baud rate, breaking pppd
    connections.
  + fix LP: #416418 - First connection attempt via F3507g fails
  + fix LP: #424603 - Not closing modems when disconnected

  [ Alexander Sack <email address hidden> ]
  * address MIR comments (LP: #410259)
    + cleanup debian/copyright
    + set debhelper compat level to 5, matching debian/control
      - add debian/compat
    + remove not used patches
      - delete debian/patches/ftbfs_hardy_glib_include_signal_h.patch
    + use proper automake lower and upper version bounds matching the
      versions referred to in debian/rules
      - update debian/control
    + don't redefine cdbs variables DEB_SOURCE, DEB_VERSION and
      DEB_UPSTREAM_VERSION; in turn move DEB_MAJOR_VERSION definition
      below the cdbs includes and use DEB_SOURCE_PACKAGE in GET_SOURCE
      function
      - update debian/rules
    + document why we touch aclocal.m4 in pre-build::
      - update debian/rules
    + use Ubuntu Network Manager Team as Maintainer and add <email address hidden>
      to Uploaders
      - update debian/control

 -- Alexander Sack <email address hidden> Mon, 14 Sep 2009 11:38:05 +0200

Changed in modemmanager (Ubuntu):
status: Fix Committed → 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.