#
1.17 |
|
04-Dec-2022 |
cheloha |
userspace: remove vestigial '?' cases from top-level getopt(3) loops
getopt(3) returns '?' when it encounters a flag not present in the in the optstring or if a flag is missing its option argument. We can handle this case with the "default" failure case with no loss of legibility. Hence, remove all the redundant "case '?':" lines.
Prompted by dlg@. With help from dlg@ and millert@.
Link: https://marc.info/?l=openbsd-tech&m=167011979726449&w=2
ok naddy@ millert@ dlg@
|
Revision tags: OPENBSD_7_1_BASE OPENBSD_7_2_BASE
|
#
1.16 |
|
24-Oct-2021 |
deraadt |
For open/openat, if the flags parameter does not contain O_CREAT, the 3rd (variadic) mode_t parameter is irrelevant. Many developers in the past have passed mode_t (0, 044, 0644, or such), which might lead future people to copy this broken idiom, and perhaps even believe this parameter has some meaning or implication or application. Delete them all. This comes out of a conversation where tb@ noticed that a strange (but intentional) pledge behaviour is to always knock-out high-bits from mode_t on a number of system calls as a safety factor, and his bewilderment that this appeared to be happening against valid modes (at least visually), but no sorry, they are all irrelevant junk. They could all be 0xdeafbeef. ok millert
|
Revision tags: OPENBSD_6_6_BASE OPENBSD_6_7_BASE OPENBSD_6_8_BASE OPENBSD_6_9_BASE OPENBSD_7_0_BASE
|
#
1.15 |
|
04-Jun-2019 |
otto |
Do nog segfault on malformed ttys(5) entry; ok kn@ jca@
|
Revision tags: OPENBSD_6_5_BASE
|
#
1.14 |
|
13-Jan-2019 |
deraadt |
Skip open of "network" ttys because they permanently retain the (large) tty buffers after that. from miod
|
Revision tags: OPENBSD_5_3_BASE OPENBSD_5_4_BASE OPENBSD_5_5_BASE OPENBSD_5_6_BASE OPENBSD_5_7_BASE OPENBSD_5_8_BASE OPENBSD_5_9_BASE OPENBSD_6_0_BASE OPENBSD_6_1_BASE OPENBSD_6_2_BASE OPENBSD_6_3_BASE OPENBSD_6_4_BASE
|
#
1.13 |
|
04-Dec-2012 |
deraadt |
remove some unnecessary sys/cdefs.h inclusions
|
Revision tags: OPENBSD_4_7_BASE OPENBSD_4_8_BASE OPENBSD_4_9_BASE OPENBSD_5_0_BASE OPENBSD_5_1_BASE OPENBSD_5_2_BASE
|
#
1.12 |
|
27-Oct-2009 |
deraadt |
rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and unmaintainable). these days, people use source. these id's do not provide any benefit, and do hurt the small install media (the 33,000 line diff is essentially mechanical) ok with the idea millert, ok dms
|
Revision tags: OPENBSD_3_8_BASE OPENBSD_3_9_BASE OPENBSD_4_0_BASE OPENBSD_4_1_BASE OPENBSD_4_2_BASE OPENBSD_4_3_BASE OPENBSD_4_4_BASE OPENBSD_4_5_BASE OPENBSD_4_6_BASE
|
#
1.11 |
|
20-Apr-2005 |
jmc |
simplify synopsis somewhat, sort options, and sync usage();
|
Revision tags: OPENBSD_3_2_BASE OPENBSD_3_3_BASE OPENBSD_3_4_BASE OPENBSD_3_5_BASE OPENBSD_3_6_BASE OPENBSD_3_7_BASE
|
#
1.10 |
|
03-Jul-2002 |
deraadt |
ansi
|
#
1.9 |
|
29-May-2002 |
deraadt |
more strlcat
|
Revision tags: OPENBSD_3_1_BASE
|
#
1.8 |
|
16-Feb-2002 |
millert |
Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.
|
Revision tags: OPENBSD_2_1_BASE OPENBSD_2_2_BASE OPENBSD_2_3_BASE OPENBSD_2_4_BASE OPENBSD_2_5_BASE OPENBSD_2_6_BASE OPENBSD_2_7_BASE OPENBSD_2_8_BASE OPENBSD_2_9_BASE OPENBSD_3_0_BASE
|
#
1.7 |
|
04-Mar-1997 |
tholo |
Initialize path earlier so we never try to use it before it has been initialized; from Time Pierce <twpierce@mail.bsd.uchicago.edu>
|
#
1.6 |
|
15-Jan-1997 |
millert |
getopt(3) returns -1 when out of args, not EOF, whee!
|
#
1.5 |
|
30-Oct-1996 |
tholo |
Really set flags...
|
Revision tags: OPENBSD_2_0_BASE
|
#
1.4 |
|
30-Aug-1996 |
deraadt |
add -p for printing; cleanup error handling
|
#
1.3 |
|
23-Jun-1996 |
deraadt |
update rcsid
|
#
1.2 |
|
21-Apr-1996 |
deraadt |
sync to netbsd 960418
|
#
1.1 |
|
18-Oct-1995 |
deraadt |
branches: 1.1.1; Initial revision
|
#
1.16 |
|
24-Oct-2021 |
deraadt |
For open/openat, if the flags parameter does not contain O_CREAT, the 3rd (variadic) mode_t parameter is irrelevant. Many developers in the past have passed mode_t (0, 044, 0644, or such), which might lead future people to copy this broken idiom, and perhaps even believe this parameter has some meaning or implication or application. Delete them all. This comes out of a conversation where tb@ noticed that a strange (but intentional) pledge behaviour is to always knock-out high-bits from mode_t on a number of system calls as a safety factor, and his bewilderment that this appeared to be happening against valid modes (at least visually), but no sorry, they are all irrelevant junk. They could all be 0xdeafbeef. ok millert
|
Revision tags: OPENBSD_6_6_BASE OPENBSD_6_7_BASE OPENBSD_6_8_BASE OPENBSD_6_9_BASE OPENBSD_7_0_BASE
|
#
1.15 |
|
04-Jun-2019 |
otto |
Do nog segfault on malformed ttys(5) entry; ok kn@ jca@
|
Revision tags: OPENBSD_6_5_BASE
|
#
1.14 |
|
13-Jan-2019 |
deraadt |
Skip open of "network" ttys because they permanently retain the (large) tty buffers after that. from miod
|
Revision tags: OPENBSD_5_3_BASE OPENBSD_5_4_BASE OPENBSD_5_5_BASE OPENBSD_5_6_BASE OPENBSD_5_7_BASE OPENBSD_5_8_BASE OPENBSD_5_9_BASE OPENBSD_6_0_BASE OPENBSD_6_1_BASE OPENBSD_6_2_BASE OPENBSD_6_3_BASE OPENBSD_6_4_BASE
|
#
1.13 |
|
04-Dec-2012 |
deraadt |
remove some unnecessary sys/cdefs.h inclusions
|
Revision tags: OPENBSD_4_7_BASE OPENBSD_4_8_BASE OPENBSD_4_9_BASE OPENBSD_5_0_BASE OPENBSD_5_1_BASE OPENBSD_5_2_BASE
|
#
1.12 |
|
27-Oct-2009 |
deraadt |
rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and unmaintainable). these days, people use source. these id's do not provide any benefit, and do hurt the small install media (the 33,000 line diff is essentially mechanical) ok with the idea millert, ok dms
|
Revision tags: OPENBSD_3_8_BASE OPENBSD_3_9_BASE OPENBSD_4_0_BASE OPENBSD_4_1_BASE OPENBSD_4_2_BASE OPENBSD_4_3_BASE OPENBSD_4_4_BASE OPENBSD_4_5_BASE OPENBSD_4_6_BASE
|
#
1.11 |
|
20-Apr-2005 |
jmc |
simplify synopsis somewhat, sort options, and sync usage();
|
Revision tags: OPENBSD_3_2_BASE OPENBSD_3_3_BASE OPENBSD_3_4_BASE OPENBSD_3_5_BASE OPENBSD_3_6_BASE OPENBSD_3_7_BASE
|
#
1.10 |
|
03-Jul-2002 |
deraadt |
ansi
|
#
1.9 |
|
29-May-2002 |
deraadt |
more strlcat
|
Revision tags: OPENBSD_3_1_BASE
|
#
1.8 |
|
16-Feb-2002 |
millert |
Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.
|
Revision tags: OPENBSD_2_1_BASE OPENBSD_2_2_BASE OPENBSD_2_3_BASE OPENBSD_2_4_BASE OPENBSD_2_5_BASE OPENBSD_2_6_BASE OPENBSD_2_7_BASE OPENBSD_2_8_BASE OPENBSD_2_9_BASE OPENBSD_3_0_BASE
|
#
1.7 |
|
04-Mar-1997 |
tholo |
Initialize path earlier so we never try to use it before it has been initialized; from Time Pierce <twpierce@mail.bsd.uchicago.edu>
|
#
1.6 |
|
15-Jan-1997 |
millert |
getopt(3) returns -1 when out of args, not EOF, whee!
|
#
1.5 |
|
30-Oct-1996 |
tholo |
Really set flags...
|
Revision tags: OPENBSD_2_0_BASE
|
#
1.4 |
|
30-Aug-1996 |
deraadt |
add -p for printing; cleanup error handling
|
#
1.3 |
|
23-Jun-1996 |
deraadt |
update rcsid
|
#
1.2 |
|
21-Apr-1996 |
deraadt |
sync to netbsd 960418
|
#
1.1 |
|
18-Oct-1995 |
deraadt |
branches: 1.1.1; Initial revision
|
#
1.15 |
|
04-Jun-2019 |
otto |
Do nog segfault on malformed ttys(5) entry; ok kn@ jca@
|
Revision tags: OPENBSD_6_5_BASE
|
#
1.14 |
|
13-Jan-2019 |
deraadt |
Skip open of "network" ttys because they permanently retain the (large) tty buffers after that. from miod
|
Revision tags: OPENBSD_5_3_BASE OPENBSD_5_4_BASE OPENBSD_5_5_BASE OPENBSD_5_6_BASE OPENBSD_5_7_BASE OPENBSD_5_8_BASE OPENBSD_5_9_BASE OPENBSD_6_0_BASE OPENBSD_6_1_BASE OPENBSD_6_2_BASE OPENBSD_6_3_BASE OPENBSD_6_4_BASE
|
#
1.13 |
|
04-Dec-2012 |
deraadt |
remove some unnecessary sys/cdefs.h inclusions
|
Revision tags: OPENBSD_4_7_BASE OPENBSD_4_8_BASE OPENBSD_4_9_BASE OPENBSD_5_0_BASE OPENBSD_5_1_BASE OPENBSD_5_2_BASE
|
#
1.12 |
|
27-Oct-2009 |
deraadt |
rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and unmaintainable). these days, people use source. these id's do not provide any benefit, and do hurt the small install media (the 33,000 line diff is essentially mechanical) ok with the idea millert, ok dms
|
Revision tags: OPENBSD_3_8_BASE OPENBSD_3_9_BASE OPENBSD_4_0_BASE OPENBSD_4_1_BASE OPENBSD_4_2_BASE OPENBSD_4_3_BASE OPENBSD_4_4_BASE OPENBSD_4_5_BASE OPENBSD_4_6_BASE
|
#
1.11 |
|
20-Apr-2005 |
jmc |
simplify synopsis somewhat, sort options, and sync usage();
|
Revision tags: OPENBSD_3_2_BASE OPENBSD_3_3_BASE OPENBSD_3_4_BASE OPENBSD_3_5_BASE OPENBSD_3_6_BASE OPENBSD_3_7_BASE
|
#
1.10 |
|
03-Jul-2002 |
deraadt |
ansi
|
#
1.9 |
|
29-May-2002 |
deraadt |
more strlcat
|
Revision tags: OPENBSD_3_1_BASE
|
#
1.8 |
|
16-Feb-2002 |
millert |
Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.
|
Revision tags: OPENBSD_2_1_BASE OPENBSD_2_2_BASE OPENBSD_2_3_BASE OPENBSD_2_4_BASE OPENBSD_2_5_BASE OPENBSD_2_6_BASE OPENBSD_2_7_BASE OPENBSD_2_8_BASE OPENBSD_2_9_BASE OPENBSD_3_0_BASE
|
#
1.7 |
|
04-Mar-1997 |
tholo |
Initialize path earlier so we never try to use it before it has been initialized; from Time Pierce <twpierce@mail.bsd.uchicago.edu>
|
#
1.6 |
|
15-Jan-1997 |
millert |
getopt(3) returns -1 when out of args, not EOF, whee!
|
#
1.5 |
|
30-Oct-1996 |
tholo |
Really set flags...
|
Revision tags: OPENBSD_2_0_BASE
|
#
1.4 |
|
30-Aug-1996 |
deraadt |
add -p for printing; cleanup error handling
|
#
1.3 |
|
23-Jun-1996 |
deraadt |
update rcsid
|
#
1.2 |
|
21-Apr-1996 |
deraadt |
sync to netbsd 960418
|
#
1.1 |
|
18-Oct-1995 |
deraadt |
branches: 1.1.1; Initial revision
|
#
1.14 |
|
13-Jan-2019 |
deraadt |
Skip open of "network" ttys because they permanently retain the (large) tty buffers after that. from miod
|
Revision tags: OPENBSD_5_3_BASE OPENBSD_5_4_BASE OPENBSD_5_5_BASE OPENBSD_5_6_BASE OPENBSD_5_7_BASE OPENBSD_5_8_BASE OPENBSD_5_9_BASE OPENBSD_6_0_BASE OPENBSD_6_1_BASE OPENBSD_6_2_BASE OPENBSD_6_3_BASE OPENBSD_6_4_BASE
|
#
1.13 |
|
04-Dec-2012 |
deraadt |
remove some unnecessary sys/cdefs.h inclusions
|
Revision tags: OPENBSD_4_7_BASE OPENBSD_4_8_BASE OPENBSD_4_9_BASE OPENBSD_5_0_BASE OPENBSD_5_1_BASE OPENBSD_5_2_BASE
|
#
1.12 |
|
27-Oct-2009 |
deraadt |
rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and unmaintainable). these days, people use source. these id's do not provide any benefit, and do hurt the small install media (the 33,000 line diff is essentially mechanical) ok with the idea millert, ok dms
|
Revision tags: OPENBSD_3_8_BASE OPENBSD_3_9_BASE OPENBSD_4_0_BASE OPENBSD_4_1_BASE OPENBSD_4_2_BASE OPENBSD_4_3_BASE OPENBSD_4_4_BASE OPENBSD_4_5_BASE OPENBSD_4_6_BASE
|
#
1.11 |
|
20-Apr-2005 |
jmc |
simplify synopsis somewhat, sort options, and sync usage();
|
Revision tags: OPENBSD_3_2_BASE OPENBSD_3_3_BASE OPENBSD_3_4_BASE OPENBSD_3_5_BASE OPENBSD_3_6_BASE OPENBSD_3_7_BASE
|
#
1.10 |
|
03-Jul-2002 |
deraadt |
ansi
|
#
1.9 |
|
29-May-2002 |
deraadt |
more strlcat
|
Revision tags: OPENBSD_3_1_BASE
|
#
1.8 |
|
16-Feb-2002 |
millert |
Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.
|
Revision tags: OPENBSD_2_1_BASE OPENBSD_2_2_BASE OPENBSD_2_3_BASE OPENBSD_2_4_BASE OPENBSD_2_5_BASE OPENBSD_2_6_BASE OPENBSD_2_7_BASE OPENBSD_2_8_BASE OPENBSD_2_9_BASE OPENBSD_3_0_BASE
|
#
1.7 |
|
04-Mar-1997 |
tholo |
Initialize path earlier so we never try to use it before it has been initialized; from Time Pierce <twpierce@mail.bsd.uchicago.edu>
|
#
1.6 |
|
15-Jan-1997 |
millert |
getopt(3) returns -1 when out of args, not EOF, whee!
|
#
1.5 |
|
30-Oct-1996 |
tholo |
Really set flags...
|
Revision tags: OPENBSD_2_0_BASE
|
#
1.4 |
|
30-Aug-1996 |
deraadt |
add -p for printing; cleanup error handling
|
#
1.3 |
|
23-Jun-1996 |
deraadt |
update rcsid
|
#
1.2 |
|
21-Apr-1996 |
deraadt |
sync to netbsd 960418
|
#
1.1 |
|
18-Oct-1995 |
deraadt |
branches: 1.1.1; Initial revision
|
Revision tags: OPENBSD_5_3_BASE OPENBSD_5_4_BASE OPENBSD_5_5_BASE OPENBSD_5_6_BASE OPENBSD_5_7_BASE OPENBSD_5_8_BASE OPENBSD_5_9_BASE OPENBSD_6_0_BASE OPENBSD_6_1_BASE OPENBSD_6_2_BASE
|
#
1.13 |
|
04-Dec-2012 |
deraadt |
remove some unnecessary sys/cdefs.h inclusions
|
Revision tags: OPENBSD_4_7_BASE OPENBSD_4_8_BASE OPENBSD_4_9_BASE OPENBSD_5_0_BASE OPENBSD_5_1_BASE OPENBSD_5_2_BASE
|
#
1.12 |
|
27-Oct-2009 |
deraadt |
rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and unmaintainable). these days, people use source. these id's do not provide any benefit, and do hurt the small install media (the 33,000 line diff is essentially mechanical) ok with the idea millert, ok dms
|
Revision tags: OPENBSD_3_8_BASE OPENBSD_3_9_BASE OPENBSD_4_0_BASE OPENBSD_4_1_BASE OPENBSD_4_2_BASE OPENBSD_4_3_BASE OPENBSD_4_4_BASE OPENBSD_4_5_BASE OPENBSD_4_6_BASE
|
#
1.11 |
|
20-Apr-2005 |
jmc |
simplify synopsis somewhat, sort options, and sync usage();
|
Revision tags: OPENBSD_3_2_BASE OPENBSD_3_3_BASE OPENBSD_3_4_BASE OPENBSD_3_5_BASE OPENBSD_3_6_BASE OPENBSD_3_7_BASE
|
#
1.10 |
|
03-Jul-2002 |
deraadt |
ansi
|
#
1.9 |
|
29-May-2002 |
deraadt |
more strlcat
|
Revision tags: OPENBSD_3_1_BASE
|
#
1.8 |
|
16-Feb-2002 |
millert |
Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.
|
Revision tags: OPENBSD_2_1_BASE OPENBSD_2_2_BASE OPENBSD_2_3_BASE OPENBSD_2_4_BASE OPENBSD_2_5_BASE OPENBSD_2_6_BASE OPENBSD_2_7_BASE OPENBSD_2_8_BASE OPENBSD_2_9_BASE OPENBSD_3_0_BASE
|
#
1.7 |
|
04-Mar-1997 |
tholo |
Initialize path earlier so we never try to use it before it has been initialized; from Time Pierce <twpierce@mail.bsd.uchicago.edu>
|
#
1.6 |
|
15-Jan-1997 |
millert |
getopt(3) returns -1 when out of args, not EOF, whee!
|
#
1.5 |
|
30-Oct-1996 |
tholo |
Really set flags...
|
Revision tags: OPENBSD_2_0_BASE
|
#
1.4 |
|
30-Aug-1996 |
deraadt |
add -p for printing; cleanup error handling
|
#
1.3 |
|
23-Jun-1996 |
deraadt |
update rcsid
|
#
1.2 |
|
21-Apr-1996 |
deraadt |
sync to netbsd 960418
|
#
1.1 |
|
18-Oct-1995 |
deraadt |
branches: 1.1.1; Initial revision
|