dnsmasq init file has incorrect DNSMASQ_INTERFACE reference

[Paul Smith]

Binary package hint: dnsmasq

Looking at the /etc/init.d/dnsmasq file I see a bug; in the start() function it says:

                ${DNSMASQ_INTERFACE:+ $DNSMASQ_INTERFACES} \

This is wrong; the variable names should both be the same. Assuming the right variable name is DNSMASQ_INTERFACES (plural) for example this should be:

                ${DNSMASQ_INTERFACES:+ $DNSMASQ_INTERFACES} \

~$ lsb_release -rd
Description: Ubuntu 10.10
Release: 10.10
~$ apt-cache policy dnsmasq
dnsmasq:
  Installed: 2.55-1
  Candidate: 2.55-1
  Version table:
 *** 2.55-1 0
        500 http://us.archive.ubuntu.com/ubuntu/ maverick/universe amd64 Packages
        100 /var/lib/dpkg/status

[Simon Kelley (simon-thekelleys)]

To Ubuntu triagers: This is a real bug, but it only affects code which
provides compatibility with very old (pre-Ubuntu) Debian installations
which might have interface configuration in /etc/default/dnsmasq. The
accepted place for such configuration has always been /etc/dnsmasq.conf
during the entire life of Ubuntu, so this is very unlikely to bite on
any Ubuntu installation. I'll fix it in the next Debian package.

Cheers,

Simon.

[Launchpad Janitor (janitor)]

This bug was fixed in the package dnsmasq - 2.57-1

---------------
dnsmasq (2.57-1) unstable; urgency=low

   * New upstream.
   * Fix typos in example config file. (closes: #606615)
   * Bump standards-version to 3.9.1
   * Add noidn DEB_BUILD_OPTIONS flag.
   * Don't complain about extra command line arguments if
     they are empty, as this breaks libvirt. (closes: #613915)

dnsmasq (2.56-1) unstable; urgency=low

   * New upstream.
   * Die if non-option args present on the command-line. (closes: #589885)
   * Tighten up use of IGNORE_RESOLVCONF in initscript. (closes: #575345)
   * Update URL of ISC's explanation of dhcp-authoritative in the example
     configuration file. (closes: #604870)
   * Cosmetic changes to dnsmasq.conf.example. (closes: #598790)
   * More dnsmasq.conf.example fixes. (closes: #606615)
   * Add other resolv.conf locations to FILES section of the manual
     page. (closes: #603505)
   * Clarify configuration for static IP addresses in the absence of
     resolvconf in the Debian readme file. (closes: #604035)
   * Fix handling of obsolete DNSMASQ_INTERFACE and DNSMASQ_EXCEPT
     variables in /etc/default/dnsmasq. (LP: #691329)
   * Provide debian/source/format.
 -- Dave Walker <email address hidden> Sun, 20 Feb 2011 02:05:50 +0000