Hello community,

here is the log from the commit of package sysconfig for 
openSUSE:12.1:Update:Test checked in at 2011-12-19 18:24:13
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:12.1:Update:Test/sysconfig (Old)
 and      /work/SRC/openSUSE:12.1:Update:Test/.sysconfig.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "sysconfig", Maintainer is "m...@suse.com"

Changes:
--------
--- /work/SRC/openSUSE:12.1:Update:Test/sysconfig/sysconfig.changes     
2011-12-19 18:24:18.000000000 +0100
+++ /work/SRC/openSUSE:12.1:Update:Test/.sysconfig.new/sysconfig.changes        
2011-12-19 18:24:18.000000000 +0100
@@ -1,0 +2,11 @@
+Mon Dec 19 09:41:09 UTC 2011 - m...@suse.com
+
+- Fixed to quote config / interface variables in ifservices script
+  and cleaned up content of the ESSID which gets appended to them
+  by NetworkManager dispatcher hook (bnc#735394, CVE-2011-4182).
+  Fixed also to return proper exit code 0 in NM dispatcher hooks.
+- Changed to call ip addr flush in ifdown, but after ip link set
+  down as it does not cause ipv6 sysctl tree side effects then
+  at least with more recent kernels (bnc#580018,bnc#559170).
+
+------------------------------------------------------------------

Old:
----
  sysconfig-nm-online-timeout-0-default.patch

New:
----
  0001-sysconfig-nm-online-timeout-0-default.patch
  0002-Fixed-order-of-addr-flush-and-link-down-in-ifdown.patch
  0003-CVE-2011-4182-fixed-quoting-in-ifservices-script.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ sysconfig.spec ++++++
--- /var/tmp/diff_new_pack.JzMu8m/_old  2011-12-19 18:24:19.000000000 +0100
+++ /var/tmp/diff_new_pack.JzMu8m/_new  2011-12-19 18:24:19.000000000 +0100
@@ -15,24 +15,23 @@
 # Please submit bugfixes or comments via http://bugs.opensuse.org/
 #
 
-# norootforbuild
-
 
 Name:           sysconfig
 Version:        0.75.4
 Release:        0
 Summary:        The sysconfig scheme and traditional network scripts
-Url:            http://gitorious.org/opensuse/sysconfig
+License:        GPL-2.0+
 Group:          System/Base
-License:        GPLv2+
-AutoReqProv:    on
+Url:            http://gitorious.org/opensuse/sysconfig
 PreReq:         %fillup_prereq %insserv_prereq textutils fileutils gawk sed 
grep
 # we may create these automatically from rpm later
 Provides:       sysvinit(network)
 Requires:       iproute2 dbus-1 procps
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 Source:         %name-%version.tar.bz2
-Patch1:         sysconfig-nm-online-timeout-0-default.patch
+Patch1:         0001-sysconfig-nm-online-timeout-0-default.patch
+Patch2:         0002-Fixed-order-of-addr-flush-and-link-down-in-ifdown.patch
+Patch3:         0003-CVE-2011-4182-fixed-quoting-in-ifservices-script.patch
 
 %description
 This package provides the SuSE system configuration scheme and
@@ -53,6 +52,8 @@
 %prep
 %setup -n sysconfig-%{version}
 %patch1 -p1
+%patch2 -p1
+%patch3 -p1
 
 %build
 autoreconf --force --install

++++++ 0001-sysconfig-nm-online-timeout-0-default.patch ++++++
>From 578a4f46906883e376ec36261bdaee4e823421da Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Mon, 10 Oct 2011 13:13:49 +0200
Subject: [PATCH] Changed NM_ONLINE_TIMEOUT default to 0

Changed NM_ONLINE_TIMEOUT default back to 0 to avoid
unneeded waiting for NM interfaces at boot (bnc#722304)

Signed-off-by: Marius Tomaschewski <m...@suse.com>
---
 config/sysconfig.config-network |    4 ++--
 sysconfig.spec.in               |   18 ------------------
 2 files changed, 2 insertions(+), 20 deletions(-)

diff --git a/config/sysconfig.config-network b/config/sysconfig.config-network
index 1f8d05a..38e3eb1 100644
--- a/config/sysconfig.config-network
+++ b/config/sysconfig.config-network
@@ -226,7 +226,7 @@ IFPLUGD_OPTIONS="-f -I -b"
 NETWORKMANAGER=no
 
 ## Type:       int
-## Default:    30
+## Default:    0
 #
 # When using NetworkManager you may define a timeout to wait for NetworkManager
 # to connect in /etc/init.d/network(-remotefs) script.  Other network services
@@ -234,7 +234,7 @@ NETWORKMANAGER=no
 # 
 # This variable has no effect if NETWORKMANAGER=no.
 #
-NM_ONLINE_TIMEOUT="30"
+NM_ONLINE_TIMEOUT="0"
 
 ## Type:        string
 ## Default:     "dns-resolver dns-bind ntp-runtime nis"
diff --git a/sysconfig.spec.in b/sysconfig.spec.in
index f559763..b64d677 100644
--- a/sysconfig.spec.in
+++ b/sysconfig.spec.in
@@ -119,13 +119,6 @@ if [ ${1:-0} -gt 1 ]; then
        if [ ! -f etc/sysconfig/network/scripts/move_shm_sysconfig.sh ] ; then
                touch etc/sysconfig/network/__move_shm_sysconfig__
        fi
-       # set a mark when updating from NM_ONLINE_TIMEOUT=0
-       eval NM_ONLINE_TIMEOUT='' \
-               `grep -s '^[[:space:]]*NM_ONLINE_TIMEOUT=' \
-               var/adm/fillup-templates/sysconfig.config-network`
-       if [ "x$NM_ONLINE_TIMEOUT" = "x0" ] ; then
-               touch etc/sysconfig/network/__nm_online_timeout__
-       fi
 fi
 
 %post 
@@ -142,17 +135,6 @@ fi
 #
 %{fillup_and_insserv -fY network}
 %{fillup_and_insserv -fY network-remotefs}
-# remove first, we need the new default value
-sysconfig_remove_and_set network/dhcp DHCLIENT_TIMEOUT
-# remove first when NM_ONLINE_TIMEOUT was 0 in old template
-if [ -f etc/sysconfig/network/__nm_online_timeout__ ] ; then
-       rm -f etc/sysconfig/network/__nm_online_timeout__
-       eval NM_ONLINE_TIMEOUT='' \
-               `grep -s '^[[:space:]]*NM_ONLINE_TIMEOUT=' \
-               etc/sysconfig/network/config`
-       [ "x$NM_ONLINE_TIMEOUT" = "x0" ] && \
-       sysconfig_remove_and_set network/config NM_ONLINE_TIMEOUT
-fi
 %{fillup_only -dns dhcp network network}
 %{fillup_only -dns config network network}
 /sbin/ldconfig
-- 
1.7.3.4

++++++ 0002-Fixed-order-of-addr-flush-and-link-down-in-ifdown.patch ++++++
>From d844f0ceb913a60e2c88b1097c98aa0b4486288f Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Fri, 18 Nov 2011 13:47:26 +0100
Subject: [PATCH] Fixed order of addr flush and link down in ifdown

Changed the order of ip addr flush and ip link set down calls
in ifdown as it does not trigger the ipv6 sysctl tree removal
(bnc#580018,bnc#559170).

Signed-off-by: Marius Tomaschewski <m...@suse.com>
---
 scripts/ifup |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/ifup b/scripts/ifup
index bd6e83a..409a925 100755
--- a/scripts/ifup
+++ b/scripts/ifup
@@ -949,8 +949,8 @@ case "$BOOTPROTO$SKIP_MAIN_PART" in
                                # Calling 'ip' if there is no interface (ifdown 
called from udev for
                                # remove event) would trigger automatic module 
loading (Bug 199456)
                                if [ -d /sys/class/net/$INTERFACE ] ; then
-                                       ip addr flush dev $INTERFACE &>/dev/null
                                        ip link set dev $INTERFACE down 
&>/dev/null
+                                       ip addr flush dev $INTERFACE &>/dev/null
                                fi
                                retcode=0 # $?
                        ;;
@@ -1134,8 +1134,8 @@ case "$BOOTPROTO$SKIP_MAIN_PART" in
                                # Calling 'ip' if there is no interface (ifdown 
called from udev for
                                # remove event) would trigger automatic module 
loading (Bug 199456)
                                if [ -d /sys/class/net/$INTERFACE ] ; then
-                                       ip addr flush dev $INTERFACE &>/dev/null
                                        ip link set dev $INTERFACE down 
&>/dev/null
+                                       ip addr flush dev $INTERFACE &>/dev/null
                                fi
                                retcode=0 # $?
                                ;;
-- 
1.7.3.4

++++++ 0003-CVE-2011-4182-fixed-quoting-in-ifservices-script.patch ++++++
>From 74f224c74c2c463365b0d39c14117870ce5776d5 Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Fri, 16 Dec 2011 16:03:15 +0100
Subject: [PATCH] CVE-2011-4182 - fixed quoting in ifservices script

Fixed to quote config / interface variables in ifservices script
and cleaned up content of the ESSID which gets appended to them
by NetworkManager dispatcher hook (bnc#735394, CVE-2011-4182).
Fixed also to return proper exit code 0 in NM dispatcher hooks.

Signed-off-by: Marius Tomaschewski <m...@suse.com>
---
 scripts/ifup-services       |   10 +++++-----
 scripts/netcontrol_services |    6 ++++--
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/scripts/ifup-services b/scripts/ifup-services
index 2047d34..8a3b083 100755
--- a/scripts/ifup-services
+++ b/scripts/ifup-services
@@ -95,15 +95,15 @@ done
 ######################################################################
 # check presence of configuration file and source it
 #
-test -f ./ifcfg-$CONFIG && . ./ifcfg-$CONFIG
+test -f "./ifcfg-$CONFIG" && . "./ifcfg-$CONFIG"
 if [ -d "ifservices-$CONFIG" ] ; then
-       cd ifservices-$CONFIG
+       cd "ifservices-$CONFIG"
 elif [ -d "ifservices-$INTERFACE" ] ; then
-       cd ifservices-$INTERFACE
+       cd "ifservices-$INTERFACE"
 elif [ -d "ifservices-${INTERFACE%%-*}" ] ; then
-       cd ifservices-${INTERFACE%%-*}
+       cd "ifservices-${INTERFACE%%-*}"
 elif [ -d "ifservices" ] ; then
-       cd ifservices
+       cd "ifservices"
 else
        debug "No services to handle for '$CONFIG $INTERFACE'"
        exit 0
diff --git a/scripts/netcontrol_services b/scripts/netcontrol_services
index 928f466..d0a55b7 100755
--- a/scripts/netcontrol_services
+++ b/scripts/netcontrol_services
@@ -28,10 +28,12 @@
 # Note that services are stopped always _after_ the interface is down. Stopping
 # services earlier would require a change in NetworkManager itself.
 
-cd /etc/sysconfig/network/ || exit
+cd /etc/sysconfig/network/ || exit 0
 test -r ./config && . ./config
 test -r scripts/functions && . scripts/functions
 
-E="`iwconfig ${1} 2>/dev/null | sed -n 's/^.*ESSID:\"\([^\"]*\)\".*$/\1/p'`"
+E=`iwconfig "${1}" 2>/dev/null | \
+       sed -n 's/^.*ESSID:\"\([^\"]*\)\".*$/\1/p' | \
+       sed -e 's/[^abcdefghijklmnopqrstuvwxyz0123456789=._-]/_/gi'`
 info_mesg "calling 'if${2}-services ${1}${E:+-$E}'"
 scripts/if${2}-services "${1}${E:+-$E}"
-- 
1.7.3.4

-- 
To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org
For additional commands, e-mail: opensuse-commit+h...@opensuse.org

Reply via email to