Package: netperf
Version: 2.6.0-2
Severity: normal
Tags: patch

Dear Maintainer,

In the init.d script, the '$' character is missing in front of the
NETSERVER_ENABLE variable. As a consequence, this variable is not
expanded and the check is never performed.

Please, apply the attached patch.

Regards,

Simon

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (1001, 'unstable'), (600, 'experimental'), (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.11-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages netperf depends on:
ii  libc6  2.17-97

netperf recommends no packages.

netperf suggests no packages.

-- Configuration Files:
/etc/default/netperf changed [not included]
/etc/init.d/netperf changed [not included]

-- no debconf information
>From 4f58328e01e2766d3411acc3f7ed7ac4ac4963cd Mon Sep 17 00:00:00 2001
From: Simon Guinot <simon.gui...@sequanux.org>
Date: Fri, 13 Dec 2013 11:31:02 +0100
Subject: [PATCH] Fix NETSERVER_ENABLE check in init.d script

This patch fixes the NETSERVER_ENABLE variable expansion by adding the
missing leading character '$'.

Signed-off-by: Simon Guinot <simon.gui...@sequanux.org>
---
 debian/init.d | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/init.d b/debian/init.d
index 4744a73..bd80b05 100644
--- a/debian/init.d
+++ b/debian/init.d
@@ -31,7 +31,7 @@ test -f $DAEMON || exit 0
 
 case "$1" in
   start)
-	if [ x"NETSERVER_ENABLE" = xno ]
+	if [ x"$NETSERVER_ENABLE" = xno ]
 	then
 		log_warning_msg "netperf daemon not enabled in /etc/default/netperf, not starting..."
 		return 0
-- 
1.8.5.1

>From 4f58328e01e2766d3411acc3f7ed7ac4ac4963cd Mon Sep 17 00:00:00 2001
From: Simon Guinot <simon.gui...@sequanux.org>
Date: Fri, 13 Dec 2013 11:31:02 +0100
Subject: [PATCH] Fix NETSERVER_ENABLE check in init.d script

This patch fixes the NETSERVER_ENABLE variable expansion by adding the
missing leading character '$'.

Signed-off-by: Simon Guinot <simon.gui...@sequanux.org>
---
 debian/init.d | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/init.d b/debian/init.d
index 4744a73..bd80b05 100644
--- a/debian/init.d
+++ b/debian/init.d
@@ -31,7 +31,7 @@ test -f $DAEMON || exit 0
 
 case "$1" in
   start)
-	if [ x"NETSERVER_ENABLE" = xno ]
+	if [ x"$NETSERVER_ENABLE" = xno ]
 	then
 		log_warning_msg "netperf daemon not enabled in /etc/default/netperf, not starting..."
 		return 0
-- 
1.8.5.1

Reply via email to