Bug 69529 - adsl-setup creates a new setup when user wants to edit one
Summary: adsl-setup creates a new setup when user wants to edit one
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: rp-pppoe
Version: 7.3
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Aaron Brown
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2002-07-22 23:55 UTC by Enrique Perez-Terron
Modified: 2007-04-18 16:44 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2002-07-22 23:55:36 UTC
Embargoed:


Attachments (Terms of Use)

Description Enrique Perez-Terron 2002-07-22 23:55:32 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:0.9.4) Gecko/20011126
Netscape6/6.2.1

Description of problem:
If the user has several setups (ppp0, ppp1, ...) and he wants to edit
ppp1, the script reports an error when comparing the user-specified
device ppp1 agains each existing but different device, e.g. ppp0.
Then the script creates a new setup if the specified one is not found.

Version-Release number of selected component (if applicable):3.4-3


How reproducible:
Always

Steps to Reproduce:
1.Create two or more files /etc/sysconfig/network-scripts/ifcfg-ppp?
  where '?' is e.g. 0, 1, 3, 5. Make sure the files contain DEVICE=ppp?
  as in the file name. Also include TYPE=xDSL and NAME=something lines.
2.Run adsl-setup. The values of DEVICE and NAME are listed for each file.
Specify the last device.
.
3.Repeat step two, but specify a non-existing device.

	

Actual Results:  The program says the device is not found, and asks you to enter 
a correct device. It repeats this message for each device that is
different until it finds the right one.Then the program lets you
edit the specified device

If you specified a non-existing device instead, the program creates one.

Expected Results:  If the device exists, it should not complain.
If it does not exist it should remain in the loop until
you either say 'n' for new (as instructed) or specify 
an exisitng device.

Additional info:

I have a patch against version rp-pppoe-3.4-3:
--- adsl-setup.rawhide.3.4-6    Sun Jun  9 19:05:29 2002
+++ adsl-setup  Tue Jul 23 00:45:01 2002
@@ -149,9 +149,11 @@
            fi
            for i in $devices ; do
                [ "$dsl_device" = "$i" ] && break
-               $ECHO "Device '$dsl_device' is not found in the list, please
choose the correct one"
            done
-           break
+           if [ "$dsl_device" = "$i" ] ; then
+               break
+           fi
+           $ECHO "Device '$dsl_device' is not found in the list, please choose
the correct one"
        fi
     done
 else

Comment 1 Ngo Than 2002-08-01 16:10:36 UTC
it's fixed in rp-pppoe-3.4-7. Thanks for your report.


Note You need to log in before you can comment on or make changes to this bug.