Bug 73638

Summary: ipfw(8): Clarify syntax for use of tables with ipfw
Product: Documentation Reporter: Mark Rowlands <mark.rowlands>
Component: Books & ArticlesAssignee: Tom Rhodes <trhodes>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Mark Rowlands 2004-11-07 10:10:26 UTC
man 8 ipfw describes syntax for table use in ipfw like this:-

addr: [not] {any | me | table(number[,value]) | addr-list | addr-set}
..
table(number[,value])

Running under bash / sh / csh 
ipfw add 2 deny ip from table(1) to me in via xl1 generates a syntax error. 

the table(x,y) entry seems to require quotes under most shells.

I feel the man page should indicate this

Fix: 

Add warning to man page that "table(x,y)" may well be required
dependant on particular shell...
How-To-Repeat: sh# ipfw add 201 deny ip from any to table(2) in via xl1
Syntax error: "(" unexpected

csh# ipfw add 201 deny ip from any to table(2) in via xl1
Badly placed ()'s.

bash# ipfw add 201 deny ip from any to table(2) in via xl1
Syntax error: "(" unexpected
Comment 1 dfilter service freebsd_committer freebsd_triage 2006-10-09 23:12:30 UTC
trhodes     2006-10-09 22:12:09 UTC

  FreeBSD src repository

  Modified files:
    sbin/ipfw            ipfw.8 
  Log:
  Add a note about rule syntax compared to the shell used so users do not get
  frustraited when:
  ipfw add 201 deny ip from any to table(2) in via xl1
  returns "Badly placed ( )'s"
  
  PR:     73638
  
  Revision  Changes    Path
  1.197     +4 -0      src/sbin/ipfw/ipfw.8
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 2 Tom Rhodes freebsd_committer freebsd_triage 2006-10-09 23:12:47 UTC
State Changed
From-To: open->patched

I added a note to the end of the manual page, under WARNINGS; however, 
I'm not sure how I feel about each manual page having this type of note. 


Comment 3 Tom Rhodes freebsd_committer freebsd_triage 2006-10-09 23:12:47 UTC
Responsible Changed
From-To: freebsd-doc->trhodes

Over to me.
Comment 4 Tom Rhodes freebsd_committer freebsd_triage 2006-11-03 09:00:54 UTC
State Changed
From-To: patched->closed

Changes merged into RELENG_6, thanks!
Comment 5 dfilter service freebsd_committer freebsd_triage 2006-11-03 09:00:55 UTC
trhodes     2006-11-03 09:00:19 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_6)
    sbin/ipfw            ipfw.8 
  Log:
  Paritially sync with CURRENT, kills hard sentence breaks, fixes some markup.
  
  MFC: 1.197
    Add a note about rule syntax compared to the shell used so users do not get
    frustraited when:
    ipfw add 201 deny ip from any to table(2) in via xl1
    returns "Badly placed ( )'s"
  
    PR:     73638
  
  Approved by:    re (hrs)
  
  Revision    Changes    Path
  1.175.2.10  +9 -3      src/sbin/ipfw/ipfw.8
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"