uncorrect verifycation static routes in /etc/network/if-up.d/20static-routes (ifupdown-extra)

Bug #631533 reported by Max V. Zhurkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ifupdown-extra (Debian)
New
Undecided
Unassigned
ifupdown-extra (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: ifupdown-extra

file /etc/network/if-up.d/20static-routes contain next code:
if [ -n "$interface" ] && [ -n "$network" ] && [ -n "$netmask" ] && [ "$gateway" = "reject" ] ; then
...
fi
what meens only "reject" routes to be add to routing table
other correct string must cause "ERROR: Incorrect line for ..."
P.S. 10.4.1
P.P.S. bug from upstream, exist in squeeze

Related branches

description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.2 KiB)

This bug was fixed in the package ifupdown-extra - 0.17

---------------
ifupdown-extra (0.17) unstable; urgency=low

  * Move the network-cable test from /etc/network/if-pre-up.d/ to
   /etc/network/if-up.d/ so that it can behave properly and warn the sysadmin
   once the network cable is up.

ifupdown-extra (0.16) unstable; urgency=low

  * if-up-scripts/check-duplicate-ip:
    - Location of ethtool has changed: adjust to the new location but fallback
      to the old one if it is still there.
    - Make it possible to use arping's arping which: is located in another
      directory, interprets parameters differently and does not support -q.
    Thanks to Cristian Ionescu-Idbohrn for bringing up this issue
    and providing a patch which I base the changes on (Closes: #614056)
    - Do not try to determine the ip address for interfaces we are not goint
      to arping to.
    - Add some warnings for unexpected events that prevent the script
      from working.
  * if-up-scripts/check-network-cable: Location of ethtool has changed,
    adjust to the new location but fallback to the old one if it is
    still there.

ifupdown-extra (0.15) unstable; urgency=medium

  * if-up-scripts/static-routes:
      - Fix typo that prevent the script from adding routes as it expected them
        to have 'reject' when they shouldn't. Thanks to Mathieu Parent and
        to Petru Ratiu for the patches. (Closes: #613632, #458395) (LP: #631533)
  * scripts/network-test:
      - Fix call to mktemp to use --tmpdir so that the script does not break if
        run in a non-writable directory (Closes: #541619)
      - Trap interrupts and abort under some circunstances, such as a
        Ctrl+C (LP: #450753)
  * debian/ifupdown-extra.preinst: Rename the /etc/network/network-routes
    config file to /etc/network/routes (Closes: #611982)
  * if-up-scripts/check-network-cable: Make the script work when bonding
    interfaces are used, using a patch provided by Ohad Lutzky (Closes: #574333)
  * [NEW] debian/ifupdown-extra.networking-routes.init: New init.d script
    to install global static networking routes which cannot be associated
    with any interface. This is useful for admins that want to share
    the configuration file /etc/network/routes through different systems
    in which the interface name changes or for 'reject' routes.
    (Closes: #458395)
  * debian/rules:
      - Fix installation of the network-routes sample configuration file so
        that it is installed where it should be (/etc/network/routes instead of
        /etc/network/network-routes) (Closes: #611982)
      - Install the new debian/ifupdown-extra.networking-routes.init as an
        init.d script running at 'S' runlevel right after the 'networking'
        script provided by netbase.
  * debian/TODO: An item was already done.
  * debian/control: Adapt description to indicate the new features included
    in the package.
  [ Lintian fixes ]
  * debian/copyright:
     - Add proper copyright notice and use UTF-8 to put in my full name.
     - Indicate the copyright of code reused from SuSE's sysconfig
  * debian/control: Depend on 'iputils-ping | ping' instead of just ping
 -- Ubunt...

Read more...

Changed in ifupdown-extra (Ubuntu):
status: New → 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.