Errors in script /usr/lib/avahi/avahi-daemon-check-dns.sh

Bug #1870824 reported by rklemme
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
avahi (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

In line 42 and 46 Command Substitution is used wrongly because the shell expects to execute what it returns. See attached patch.

Tags: focal patch
Revision history for this message
rklemme (shortcutter) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Patch that fixes the two errors" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
tags: added: focal
Revision history for this message
Trent Lloyd (lathiat) wrote :

For focal we should actually remove this script, since nss-mdns automatically performs this behaviour and doesn't rely on the "hack" of stopping avahi-daemon to prevent resolution of .local domains via DNS.

If this error is serious we could consider backport to stable releases.

Revision history for this message
Trent Lloyd (lathiat) wrote :

(for clarity nss-mdns 0.14 does this behavior itself automatically, in 0.10 in bionic does not)

Revision history for this message
rklemme (shortcutter) wrote :

I think the script will still work most of the time so severity is not too high.

Changed in avahi (Ubuntu):
status: New → In Progress
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package avahi - 0.7-4ubuntu7

---------------
avahi (0.7-4ubuntu7) focal; urgency=medium

  * Remove avahi-daemon-check-dns.sh hack, the feature is provided by
    libnss-mdns now (lp: #1870824)
  * debian/patches/local-only-services-support.patch:
    - replaced by the upstream commited version, part of the code which
      was there to workaround a ippusbxd issue has been removed since
      the problem has been resolved in cups now

 -- Sebastien Bacher <email address hidden> Wed, 08 Apr 2020 13:43:27 +0200

Changed in avahi (Ubuntu):
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.