NetworkManager connection sharing broken

Bug #725041 reported by Juho Teperi
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
network-manager (Ubuntu)
Fix Released
High
Mathieu Trudel-Lapierre
Natty
Fix Released
High
Mathieu Trudel-Lapierre

Bug Description

Binary package hint: network-manager

Ubuntu 11.04, network-manager 0.8.3+git.20110210t152733.00327fb-0ubuntu2, dnsmasq 2.57-1

I have shared my connection using NetworkManager (IPv4 Settings -> Method: Shared to other computers), however this is not working.
I have attached related parts from syslog with NetworkManager's debug enabled.

Revision history for this message
Juho Teperi (deraen) wrote :
Revision history for this message
Juho Teperi (deraen) wrote :

Seems to be because new dnsmasq (2.56+, http://www.thekelleys.org.uk/dnsmasq/CHANGELOG) checks if there is junk on its parameters and NetworkManager is sending bogus conf-file.
Related part from nm source: http://cgit.freedesktop.org/NetworkManager/NetworkManager/tree/src/dnsmasq-manager/nm-dnsmasq-manager.c#n278

Also I tried to run dnsmasq without bogus conf file and with, here are results:

juho@juho:~$ sudo dnsmasq --keep-in-foreground
^Cjuho@juho:~$ sudo dnsmasq --keep-in-foreground --conf-file /tmp/sdfsdf.conf

dnsmasq: junk found in command line
juho@juho:~$ sudo dnsmasq --keep-in-foreground --conf-file=/tmp/sdfsdf.conf

dnsmasq: cannot read /tmp/sdfsdf.conf: No such file or directory
juho@juho:~$

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :
Changed in network-manager (Ubuntu):
status: New → Triaged
importance: Undecided → High
status: Triaged → In Progress
assignee: nobody → Mathieu Trudel-Lapierre (mathieu-tl)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package network-manager - 0.8.4~git.20110228t143901.5cdded6-0ubuntu1

---------------
network-manager (0.8.4~git.20110228t143901.5cdded6-0ubuntu1) natty; urgency=low

  * upstream snapshot 2011-02-28 14:39:01 (GMT) (LP: #726820)
    + 5cdded6b66bdc371acb29e76187b174c55c11470
    - docs: explicitly list libnm-glib objects
    - libnm-util: fix parameter name for docs generator
    - api: add missing Settings.System D-Bus interface bits
    - ifcfg-rh: get rid of LAST_CONNECT
    - core/libnm-glib: don't touch files in /etc when updating timestamps
    - release: bump version to 0.8.3.996 (0.8.4-beta2)
    - api: document more VPN connection failure codes (bgo #643324)
    - libnm-util: silently ignore non-secrets when updating connection secrets
    - libnm-util: fix updating secrets (rh #680385)
    - keyfile: fix tests for no-longer-required serial and PPP settings
    - ip6: fail activation if addrconf fails to start
    - core: don't require serial and PPP settings for mobile broadband
    - ppp: fix warning when PPP support is disabled
    - libnm-util: bump version for several recent changes
    - release: bump version to 0.8.3.995 (0.8.4-beta1)
    - release: update NEWS with changes since 0.8.2
    - libnm-util: fix updating secrets
    - core: flush IPv6 routes even if configuration failed (bgo #641333)
      (LP: #704812)
    - libnm-util: remove GObject Introspection annotations
    - gentoo: simplify OpenRC detection
    - libnm-util: add nm_connection_diff()
    - dns: fix up updating resolv.conf (rh #672282)
    - Revert "libnm-glib: return enums for NMAccessPoint flags accessors"
    - build: allow disabling PPP support at build time
    - core: fix dbus-glib crash on shutdown with systemd
    - libnm-glib: return enums for NMAccessPoint flags accessors
    - libnm-util: fix description of bluetooth setting type property
    - test: don't crash libnm-glib if there are no devices for an active
      connection
    - test: don't crash libnm-glib-test if there aren't any APs
  * debian/patches/revert_52f2295.patch: dropped, fixed upstream.
  * debian/libnm-util1.symbols: adding new symbols introduced upstream:
    - nm_connection_diff
    - nm_setting_diff
  * debian/patches/fix_sorting_connections_w_tstamp_3194cb9.patch: fix sorting
    connections following upstream commit 7d707bd, which is necessary for auto
    activations.
  * debian/patches/dnsmasq_conffile_conn_sharing.patch: don't pass a bogus conf
    file to dnsmasq, just pass no file parameter (LP: #725041)
  * debian/rules: enable build-time tests. (LP: #727395)
  * debian/patches/70_lp145653_no_sigaction_for_crashes.patch: add .pc file
    generated by quilt to POTFILES.skip, otherwise the build will fail due to
    the tests we now run detecting a localizable file missing from templates.
 -- Mathieu Trudel-Lapierre <email address hidden> Fri, 04 Mar 2011 10:53:59 -0500

Changed in network-manager (Ubuntu Natty):
status: In Progress → 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.