Connection Manager

konversation disconnects after every dhcp renewal

Bug #713146 reported by Kalle Valo
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Connection Manager
Fix Committed
Medium
Kalle Valo
connman (Ubuntu)
Fix Released
Medium
Mathieu Trudel-Lapierre

Bug Description

konversation disconnects the connection to an irc server after every dhcp renewal with connman. I checked with this tcpdump, the tcp connection is closed immeadiately (max 100 ms) after dhcp ack from the server.

Related branches

lp://staging/~cyphermox/ubuntu/natty/connman/0.69
Micah Gersten: Approve
Diff: 25752 lines (+11190/-7957)
127 files modified
AUTHORS (+4/-0)
ChangeLog (+44/-0)
Makefile.am (+20/-14)
Makefile.in (+277/-274)
Makefile.plugins (+42/-34)
README (+1/-16)
TODO (+6/-58)
acinclude.m4 (+1/-0)
configure (+211/-188)
configure.ac (+37/-26)
debian/changelog (+21/-0)
debian/control (+3/-2)
debian/manpages/connmand.8 (+10/-1)
debian/patches/04-wifi-add-a-disconnected-timer.patch (+58/-16)
debian/patches/09-disable-wifi-legacy.patch (+0/-38)
debian/patches/10-fix-dhcp-renewal-disconnect.patch (+172/-0)
debian/patches/series (+1/-1)
debian/rules (+3/-1)
doc/agent-api.txt (+30/-7)
doc/config-format.txt (+3/-1)
doc/counter-api.txt (+1/-1)
doc/ipconfig-api.txt (+1/-1)
doc/manager-api.txt (+0/-7)
doc/overview-api.txt (+11/-11)
doc/plugin-api.txt (+1/-1)
doc/service-api.txt (+12/-22)
doc/technology-api.txt (+23/-0)
gdbus/gdbus.h (+1/-1)
gdbus/mainloop.c (+1/-1)
gdbus/object.c (+18/-17)
gdbus/polkit.c (+2/-2)
gdbus/watch.c (+16/-17)
gdhcp/client.c (+322/-12)
gdhcp/common.c (+1/-0)
gdhcp/gdhcp.h (+4/-0)
gdhcp/ipv4ll.c (+139/-0)
gdhcp/ipv4ll.h (+55/-0)
gdhcp/server.c (+2/-2)
gsupplicant/dbus.c (+1/-3)
gsupplicant/gsupplicant.h (+34/-0)
gsupplicant/supplicant.c (+644/-33)
gweb/gresolv.c (+560/-93)
gweb/gweb.c (+199/-24)
gweb/gweb.h (+2/-0)
include/device.h (+2/-0)
include/dhcp.h (+0/-80)
include/inet.h (+9/-0)
include/ipconfig.h (+3/-20)
include/location.h (+3/-0)
include/network.h (+4/-3)
include/notifier.h (+1/-0)
include/ondemand.h (+0/-38)
include/provider.h (+10/-2)
include/proxy.h (+5/-0)
include/resolver.h (+2/-17)
include/service.h (+4/-0)
include/task.h (+1/-1)
include/technology.h (+6/-0)
plugins/bluetooth.c (+1/-0)
plugins/connman-nmcompat.conf (+14/-0)
plugins/dhcp.c (+0/-191)
plugins/dnsproxy.c (+0/-1371)
plugins/ethernet.c (+102/-0)
plugins/iospm.c (+1/-0)
plugins/iwmx.c (+1/-1)
plugins/mcc.h (+475/-0)
plugins/nmcompat.c (+282/-0)
plugins/ntpd.c (+2/-1)
plugins/ofono.c (+238/-65)
plugins/openconnect.c (+20/-7)
plugins/openvpn.c (+80/-56)
plugins/pacrunner.c (+163/-0)
plugins/portal.c (+45/-11)
plugins/supplicant.c (+0/-2692)
plugins/supplicant.h (+0/-41)
plugins/vpn.c (+16/-5)
plugins/vpn.h (+2/-1)
plugins/vpnc.c (+290/-0)
plugins/wifi-legacy.c (+0/-243)
plugins/wifi.c (+602/-147)
scripts/openconnect-script.c (+4/-0)
src/agent.c (+161/-1)
src/config.c (+165/-8)
src/connection.c (+8/-2)
src/connman.h (+34/-27)
src/dbus.c (+4/-0)
src/detect.c (+1/-0)
src/device.c (+48/-7)
src/dhcp.c (+183/-172)
src/dnsproxy.c (+1477/-0)
src/element.c (+8/-0)
src/inet.c (+102/-8)
src/ipconfig.c (+316/-224)
src/iptables.c (+4/-0)
src/ipv4.c (+5/-1)
src/location.c (+8/-0)
src/main.c (+7/-23)
src/manager.c (+2/-103)
src/network.c (+138/-92)
src/notifier.c (+19/-0)
src/ondemand.c (+0/-92)
src/provider.c (+230/-28)
src/proxy.c (+109/-7)
src/resolver.c (+291/-329)
src/rfkill.c (+4/-1)
src/rtnl.c (+167/-10)
src/selftest.c (+0/-43)
src/service.c (+535/-214)
src/session.c (+1/-0)
src/stats.c (+379/-51)
src/task.c (+8/-4)
src/technology.c (+258/-94)
src/tethering.c (+5/-29)
src/wpad.c (+19/-11)
test/disable-tethering (+23/-1)
test/enable-tethering (+37/-1)
test/get-services (+2/-1)
test/list-services (+3/-2)
test/monitor-services (+2/-1)
test/set-ip-method (+0/-34)
test/set-ipv4-method (+34/-0)
test/set-ipv6-method (+33/-0)
test/simple-agent (+51/-8)
test/test-manager (+9/-2)
tools/stats-ringbuffer-dump.c (+0/-408)
tools/stats-tool.c (+910/-0)
tools/web-test.c (+8/-0)
lp://staging/ubuntu/natty/connman
Kalle Valo (kvalo)
Changed in connman:
assignee: nobody → Kalle Valo (kvalo)
status: Triaged → In Progress
Revision history for this message
Kalle Valo (kvalo) wrote :

The problem was that connman reset the settings during every dchp renewal, even though there were no actual changes in the dhcp settings. konversation again has some logic to detect address changes, and this caused the disconnections from the server.

I sent an rfc patch to upstream:

http://lists.connman.net/pipermail/connman/2011-February/003770.html

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Reassigning to myself, I'm preparing the packaging to include the patch.

Changed in connman:
assignee: Kalle Valo (kvalo) → Mathieu Trudel-Lapierre (mathieu-tl)
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Woops, this was done a little wrong, should be against ubuntu package.

Changed in connman:
assignee: Mathieu Trudel-Lapierre (mathieu-tl) → Kalle Valo (kvalo)
Changed in connman (Ubuntu):
assignee: nobody → Mathieu Trudel-Lapierre (mathieu-tl)
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package connman - 0.69-0ubuntu1

---------------
connman (0.69-0ubuntu1) natty; urgency=low

  * New upstream release.
    - dhcpd.conf domain-name not transmitted to /etc/resolv.conf (LP: #604581)
    - provide NM compatiblity code for applications which request network
      status (LP: #586033)
  * debian/control: explicitly Conflict with network-manager. (LP: #659460)
  * debian/rules: enable udev and the builtin dhclient during build.
  * debian/patches/04-wifi-add-a-disconnected-timer.patch: refreshed.
  * debian/patches/09-disable-wifi-legacy.patch: dropped.
  * debian/control: drop dhcp3-client Build-Depends, it's no longer required.
  * debian/patches/10-fix-dhcp-renewal-disconnect.patch: avoid disconnections
    caused by calling dhcp_bound() when DHCP renewal results in no IP change.
    (LP: #713146)
  * debian/control: Recommend indicator-network for UI. (LP: #704867)
  * debian/manpages/connmand.8: update to add BUGS section and fix AUTHORS
    (LP: #587779)
  * debian/control: Remove binary Depends on dhcp3-client for connman.
 -- Mathieu Trudel-Lapierre <email address hidden> Tue, 15 Feb 2011 23:43:45 -0600

Changed in connman (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Kalle Valo (kvalo) wrote :

Daily builds also now have my rfc patch. This will be fixed in upstream once the element refactoring is finished, which will most likely take few weeks.

Changed in connman:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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