[Pidgin] #14755: bonjour file transfer broken when IPv6 is activated in avahi

Pidgin trac at pidgin.im
Tue Oct 30 21:45:48 EDT 2012


#14755: bonjour file transfer broken when IPv6 is activated in avahi
----------------------------------------------+-----------------------
 Reporter:  tux                               |       Owner:  datallah
     Type:  defect                            |      Status:  new
Milestone:  Patches Needing Review            |   Component:  Bonjour
  Version:  2.10.1                            |  Resolution:
 Keywords:  bonjour ipv6 file transfer avahi  |
----------------------------------------------+-----------------------

Comment (by tux):

 Changes for the second version of 'bonjour-retry-alternative-address.patch
 ':
 * Moves the xmlnode freeing to its own function "xmlnode_free_tree()"
 * Fixes a potential memleak by letting xf->streamhost point to the xf's
 current xml streamhost entry intstead of the next one (which might be
 NULL, thus resulting in xmlnode_free_tree() not freeing anything).

 ----

 I applied this patch against pidgin 2.10.6 and did a successful cross
 compilation for Windows following these steps here:
 https://developer.pidgin.im/wiki/BuildingWinPidgin#CrossCompiling

 Therefore, are there really more IPv6 tweaks needed? (also note that the
 changes done to libpurple/protocols/bonjour/bonjour_ft.c
 (add_ipv6_link_local_ifaces() ) are not that different from
 libpurple/protocols/bonjour/jabber.c (append_iface_if_linklocal() ).

 ----

 However I spotted another issue and I'm not sure whether this a regression
 of the current, proposed patch or whether that issue has been there and
 hidden before: During a test with 4 simultaneous IP addresses (1x IPv4, 1x
 link-local IPv6, 1x unique-local IPv6, 1x global IPv6) on each of two
 nodes, I was able to successfully transfer a file but now noticed that I'm
 not able to transfer subsequent files. With only one IPv4 address together
 with this patch works fine though.

-- 
Ticket URL: <https://developer.pidgin.im/ticket/14755#comment:6>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list