Bug 10907 - /etc/rc.d/init.d/ipchains enhancement
/etc/rc.d/init.d/ipchains enhancement
Status: CLOSED RAWHIDE
Product: Red Hat Linux
Classification: Retired
Component: ipchains (Show other bugs)
6.2
All Linux
medium Severity medium
: ---
: ---
Assigned To: Cristian Gafton
: FutureFeature
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2000-04-19 03:57 EDT by Mihai Ibanescu
Modified: 2008-05-01 11:37 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2000-04-19 04:08:46 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Patch to /etc/rc.d/init.d/ipchains, to properly exclude comments and new lines from $IPCHAINS_CONFIG (553 bytes, patch)
2000-04-19 04:08 EDT, Mihai Ibanescu
no flags Details | Diff

  None (edit)
Description Mihai Ibanescu 2000-04-19 03:57:10 EDT
ipchains-restore is somewhat picky when it comes to parsing the file saved
with ipchains-save. If a human intends to edit that file, almost for sure
she will insert blank lines. The default startup script for ipchains
already allows for comments in /etc/sysconfig/ipchains. But it will fail to
properly filter comments starting with spaces, or empty lines. The normal
grep would be:

grep -v "^[[:space:]]*#" | grep -v "^[[:space:]]*$"

instead of grep -v "^#"

Patch attached
Comment 1 Mihai Ibanescu 2000-04-19 04:08:59 EDT
Created attachment 207 [details]
Patch to /etc/rc.d/init.d/ipchains, to properly exclude comments and new lines from $IPCHAINS_CONFIG
Comment 2 Nalin Dahyabhai 2000-07-10 19:55:57 EDT
Incorporated directly into ipchains-1.3.9-10.  Thanks for the patch.

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