core: avahi on Linux uses incorrect address for P-t-P interface

Hi,

I sent a less knowledgeable question about avahi-daemon and
point-to-point links a few days ago,
http://lists.freedesktop.org/archives/avahi/2011-January/001969.html.
When I didn't get a response to this, I decided to build avahi from
source and step through it and see how it builds its list of interfaces
and their addresses.

This is in iface-linux.c, netlink_callback(). It looks for a RTM_NEWADDR
message, then extracts the payload of type IFA_ADDRESS.

Short story: I think it should be using the payload of type IFA_LOCAL,
not the payload of type IFA_ADDRESS.

In the VM where I was running these experiments, there are 3 interfaces
-- lo, eth0 and tun0. I printed out the IFA_ADDRESS and IFA_LOCAL for
all 3 of these; for lo and eth0 these are the same address; for tun0
(IFF_POINTOPOINT), IFA_ADDRESS is the remote end and IFA_LOCAL is the
local end.

I'm no expert on Linux rtnetlink or these IFA fields, but quoting
/usr/include/linux/if_addr.h:

/*
 * Important comment:
 * IFA_ADDRESS is prefix address, rather than local interface address.
 * It makes no difference for normally configured broadcast
 * interfaces,
 * but for point-to-point IFA_ADDRESS is DESTINATION address,
 * local address is supplied in IFA_LOCAL attribute.
 */

See also this stackoverflow question/answer:
http://stackoverflow.com/questions/4678637/what-is-difference-between-ifa-local-and-ifa-address-in-rtnetlink-linux

Does anyone know why avahi is looking for IFA_ADDRESS here, and whether
there's any reason not to use IFA_LOCAL instead?

Assuming there's not a specific reason to use IFA_ADDRESS here, I
propose the patch attached at the end of this message, which works for
me. (The bug this fixes is described in the earlier message linked above
-- avahi chooses the wrong address to associate with the P-t-P
interface, and if you enable avahi's reflector, avahi tries to call
sendmsg() using that as the source address and the kernel always,
correctly, fails the sendmsg() call with EINVAL.)

(And when/why this changed -- I was able to use avahi over P-t-P
interfaces on Linux several years ago; I don't know what avahi version I
was using at the time.)

thanks,

Matt

reflector: cache entry link-local addresses filter

Do not reflect cache entry with ipv6 link-local addresses on query.

init: suse, avahi-dnsconfd status reports dead instead of unused

Keep emacs configuration in one configuration file.

On 07.03.2011 19:04, Lennart Poettering wrote:
>
> Oh, I had assumed that .vimrc trick would allow project-wide vim
> modelines without having to edit each and every single file.
>
> Currently, every file does contain an emacs modeline at the top. It
> would be fair I guess to add a vim modeline to all those files too, even
> though it's not necessarily pretty.
Hi,
maybe it makes sense to go in the opposite direction: add .dir-locals.el
in the top directory. The settings are actually identical in all .[ch]
files.

gobject: ga_service_browser_new() should accept const gchar*

ga_service_browser_new() and ga_service_browser_new_full() should
accept const gchar*.

setproctitle: use defined() around __linux__ in #elif

socket: Still read corrupt packets from the sockets

Else, we end up with an infinite loop with 100% CPU.

http://www.avahi.org/ticket/325
https://bugzilla.redhat.com/show_bug.cgi?id=667187

man: more spelling fixes

https://bugzilla.redhat.com/show_bug.cgi?id=673812
https://bugzilla.redhat.com/show_bug.cgi?id=673810

man: spelling fixes

https://bugzilla.redhat.com/show_bug.cgi?id=673812
https://bugzilla.redhat.com/show_bug.cgi?id=673810

avahi-ui: gtk3 fix

netlink: fix wrong use of netlink flags for dump operations

The avahi-daemon uses a wrong flag combination to operate with
rtnetlink. This patch fixes the problems.

No need to set NLM_F_ACK since the dump operation already includes
the trailing NLMSG_DONE message that informs about the end of the
dump operation.

daemon: we better stay away from SIGQUIT since it is supposed to cause a core dump

systemd: update sd-daemon.[ch]

init: update the suse initscripts

Here are two patches for avahi. The avahi website is extremly slow
today, so I didn't file them.

 - avahi-fix-howl.pc.patch: fix a small issue introduced in
   c8eacc0f13f6df529f2f40961e2ddb3a45dc9f93
 - avahi-init-lsb.patch: update the suse initscripts, with a patch
   we've had for a while

Cheers,

Vincent

--
Les gens heureux ne sont pas pressés.

howl: fix small issue

Here are two patches for avahi. The avahi website is extremly slow
today, so I didn't file them.

 - avahi-fix-howl.pc.patch: fix a small issue introduced in
   c8eacc0f13f6df529f2f40961e2ddb3a45dc9f93
 - avahi-init-lsb.patch: update the suse initscripts, with a patch
   we've had for a while

Cheers,

  Vincent

--
Les gens heureux ne sont pas pressés.

systemd: update configure.ac snippet

update NEWS

bump version

client: use Ping() to test whether avahi is around

Ping() should work more reliably on systems where dbus activation of
avahi is not available.

http://avahi.org/ticket/319

systemd: pull in avahi from dbus activation indirectly to make it possible to deactivate it

systemd: update sd-daemon.[ch]

i18n: update LINGUAS

Merge remote branch 'origin/master-tx'

l10n: Updated Slovenian translation

New status: 160 messages complete with 0 fuzzies and 5 untranslated.

Transmitted-via: Transifex (www.transifex.net).

gtk: avahi build fix

Hey,

avahi-0.26.7 braucht einen kleinen fix, um mit gtk+ 2.21.8/2.90.7 zu
bauen. See attachment.

gobject: Fix avahi build with gobject-introspection

Hi Lennart,

Here's a patch to fix avahi to build with g-i 0.9.5 and later.

Cheers,

Vincent

--
Les gens heureux ne sont pas pressés.

l10n: Updated Slovenian translation.

New status: 135 messages complete with 10 fuzzies and 20 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Slovenian translation.

New status: 40 messages complete with 100 fuzzies and 25 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Preliminary faroese translation

New status: 113 messages complete with 5 fuzzies and 47 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated German (de) translation to 98%

New status: 162 messages complete with 2 fuzzies and 1 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Italian (it) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Japanese (ja) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated French (fr) translation to 99%

New status: 164 messages complete with 1 fuzzy and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Hungarian (hu) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Hungarian (hu) translation to 98%

New status: 163 messages complete with 2 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Danish (da) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Danish (da) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Italian (it) translation to 99%

New status: 164 messages complete with 1 fuzzy and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Chinese (Taiwan) (zh_TW) translation to 66%

New status: 109 messages complete with 1 fuzzy and 55 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Russian (ru) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updates to Indonesian translation

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Indonesian (id) translation to 97%

New status: 161 messages complete with 4 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Initial Indonesian translation

New status: 83 messages complete with 0 fuzzies and 82 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Swedish (sv) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: new danish translation

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Hebrew translation by Yaron Shahrabani.

New status: 102 messages complete with 37 fuzzies and 26 untranslated.

Transmitted-via: Transifex (www.transifex.net).

build-sys: require libdaemon 0.14 -- see #317

gcc: minor enum issues gcc 4.5 discovered

build-sys: bump soname

client: update libavahi-client so that it can deal with auto-activated avahi daemons

i18n: run make update-po

Merge remote branch 'origin/master-tx'

main: fix minor indenting mishap

init: fix lfs init script generation

systemd: update sd-daemon.[ch] from upśtream

daemon: unblock all signals by default

systemd: send status message when shutting down

daemon: reset signals on initialization

daemon: return successful error code when we ran successfully

daemon: make sure we never choke on SIGPIPE

systemd: enable status messages by default

systemd: properly format status message

systemd: update sd-daemon.[ch]

systemd: send status updates to the init system

systemd: simplify systemd unit files following the recent DefaultDependencies= changes

l10n: Updated Chinese (China) (zh_CN) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Ukrainian (uk) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Spanish (Castilian) (es) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

systemd: add description strings to all systemd services/sockets

build-sys: fix build for systems that have gtk2 but no gtk3

l10n: Updated Polish (pl) translation to 100%

New status: 165 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

fedora: fix typo in LSB header

fedora: fix init script dir

build-sys: fix homepage target

doc: add NEWS section for 0.6.26

build-sys: bump version/soname

core: implement packet rate limiting and enable it by default

fedora: init LSB header to init scripts

main: don't abort if daemon_close_all() cannot close valgrind fds

main: fix minor memory leak

core: fix potential crash on service name collision

If there is a service name collision and the entry group callback calls
avahi_s_entry_group_reset or avahi_s_entry_group free on the group in
question, the entries were released. This could cause a crash in
withdraw_rrset as it is walking a list of entries at this time.

The fix for this issue is to schedule a cleanup event to clean up
entries after a a short timeout (currently one second). If a cleanup
occurs for any other reason the event is cancelled.

http://avahi.org/ticket/302

core: fix counting of workstation services that are in the process of being registered

Fixes http://avahi.org/ticket/276

core: don't warn if we try to remove a querier that is already gone

This might happen if an iface goes away while we are querying.

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=548952

i18n: run make update-po

i18n: make avahi-discover translatable.

Original patch from "fujiwara"

http://avahi.org/ticket/122

avahi-browse: replace unprintable chars when printing services names

http://avahi.org/ticket/285

core: properly unescape host name before using it in the workstation service

dbus: allow root to set the host name unconditionally

core: don't check ARCOUNT to avoid incompatibility with EDNS0

Newer Bonjour implementations add EDNS0 extensions to the query packets.
We currently consider those corrupted or invalid. This patch drops the
check, and accepts them as valid.

http://avahi.org/ticket/284

iface: check for IFA_F_DEPRECATED

http://avahi.org/ticket/211

libdns_sd: implement DNSServiceQueryRecord()

Modified patch, originally from "AlfonsName".

http://avahi.org/ticket/270

avahi-ui: fix wording of the description strings of a few properties

As suggested in http://avahi.org/ticket/275

gobject: avoid assertion when using ga_entry_group_attach() improperly

http://avahi.org/ticket/304

static-services: unescape host name when using it in static service names

core: always normalize configure host/domain name

autoipd: fix debian's dhcp path

As discussed with mbiebl on IRC

daemon: bump up resource limit of open file descriptors a little

This should allow more clients to query hsot names via the simple
protocol.

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=412256

i18n: update gitignore

i18n: run make update-po

i18n: update LINGUAS