Commit Graph

40 Commits

Author SHA1 Message Date
naddy 41ce3b17e7 man pages: add missing commas between subordinate and main clauses
jmc@ dislikes a comma before "then" in a conditional, so leave those
untouched.

ok jmc@
2022-03-31 17:27:13 +00:00
naddy 08f6ba1906 Don't declare variables as "unsigned char *" that are passed to
functions that take "char *" arguments.  Where such chars are
assigned to int or passed to ctype functions, explicitly cast them
to unsigned char.

For OpenBSD's clang, -Wpointer-sign has been disabled by default,
but when the parse.y code was built elsewhere, the compiler would
complain.

With help from millert@
ok benno@ deraadt@
2021-10-15 15:01:27 +00:00
claudio 7fec5ffcad Document new reload behaviour. From Ashton Fagg
OK jmc@
2021-05-05 12:34:12 +00:00
claudio 2f8c4829bf When loading a config poll iscsid for up to 10 seconds and check if the
sessions all managed to connect to the targets.
With this slow session establishment should not cause mount errors
during startup.
Initial version from Dr Ashton Fagg ashton (at) fagg id au
OK dlg@ deraadt@
2021-04-16 14:39:33 +00:00
jmc 48c01a1f20 ce examples of "Ar arg Ar arg" with "Ar arg arg" and stop the spread; 2020-04-23 21:28:08 +00:00
deraadt 915c3f33d3 (unsigned) means (unsigned int) which on ptrdiff_t or size_t or other
larger types really is a range reduction...
Almost any cast to (unsigned) is a bug.
ok millert tb benno
2019-02-13 22:57:07 +00:00
sashan a153335958 - odd condition/test in PF lexer
(and other lexers too)

This commit rectifies earlier change:

    in the lex... even inside quotes, a \ followed by space or tab should
    expand to space or tab, and a \ followed by newline should be ignored
    (as a line continuation).  compatible with the needs of hoststated
    (which has the most strict quoted string requirements), and ifstated
    (where one commonly does line continuations in strings).

OK deraadt@, OK millert@
2018-11-01 00:18:44 +00:00
florian b75abfe50e struct sockaddr_storage is guaranteed to be large enough, no need to check
the size.
OK kn, deraadt
2018-10-15 11:30:01 +00:00
miko ed1b9eb881 replace malloc()+strlcpy() with strndup() in cmdline_symset().
"looks good" gilles@ halex@
2018-09-07 07:35:30 +00:00
krw a062aa9d6a Do for most running out of memory err() what was done for most running
out of memory log_warn(). i.e. ("%s", __func__) instead of manual
function names and redundant verbiage about which wrapper detected the
out of memory condition.

ok henning@
2018-07-11 07:39:22 +00:00
krw 6a3d55f939 No need to mention which memory allocation entry point failed (malloc,
calloc or strdup), we just need to log that we ran out of memory in a
particular function.

Recommended by florian@ and deraadt@

ok benno@ henning@ tb@
2018-07-09 12:05:10 +00:00
krw 3c12ada049 Be consistent in warn() and log_warn() usage when
running out of memory.

Next step, be correct *and* consistent.

ok dennis@ tb@ benno@ schwarze@
2018-07-08 17:15:07 +00:00
jmc 1caaaac72f remove the SECTIONS header, since a one line DESCRIPTION is a bit silly;
use a more general text for the sections, and avoid the catchup issue
that was trying to document how many there were;

ok benno rob
2018-06-18 06:04:25 +00:00
denis 8ab2018ae2 Fix an off-by-one line count when using include statements.
Thanks to otto@ for the initial diff.

OK benno@
2018-06-11 09:39:58 +00:00
krw 16a0a9064f Plug leak in error case of the common 'varset' implementations.
ok benno@
2018-04-26 14:12:19 +00:00
krw 46bca67b3a Replace hand-rolled for(;;) emptying of 'symhead' TAILQ with more
modern TAILQ_FOREACH_SAFE().

No intentional functional change.

ok millert@ bluhm@ gilles@
2017-01-05 13:53:09 +00:00
krw 54c95b7a05 Replace symset()'s hand-rolled for(;;) traversal of 'symhead' TAILQ
with more modern TAILQ_FOREACH(). This what symget() was already
doing.

Add paranoia '{}' around body of symget()'s TAILQ_FOREACH().

No intentional functional change.

ok bluhm@ otto@
2017-01-05 12:42:18 +00:00
tedu f75ac03558 stop including sys/param.h for nitems. define locally as needed.
ok natano reyk
2016-08-16 18:41:57 +00:00
benno 0c7b4ca6ee do not allow whitespace in macro names, i.e. "this is" = "a variable".
change this in all config parsers in our tree that support macros.
problem reported by sven falempin.

feedback from henning@, stsp@, deraadt@
ok florian@ mikeb@
2016-06-21 21:35:24 +00:00
deraadt ed720fabf5 claudio twice told me pledge "stdio rpath dns" is probably enough.
He'll test it next time he builds...
2015-11-11 02:55:12 +00:00
sobrado b9170857bc use file system path (.Pa) semantic markup macros where appropriate.
ok jmc@
2015-07-27 17:28:38 +00:00
deraadt b9fc9a728f Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible.  Annotate <sys/param.h> lines with their current reasons.  Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_NAME_MAX, etc.  Change
MIN() and MAX() to local definitions of MINIMUM() and MAXIMUM() where
sensible to avoid pulling in the pollution.  These are the files confirmed
through binary verification.
ok guenther, millert, doug (helped with the verification protocol)
2015-01-16 06:39:28 +00:00
jsg 41eef22fad Don't allow embedded nul characters in strings.
Fixes a pfctl crash with an anchor name containing
an embedded nul found with the afl fuzzer.

pfctl parse.y patch from and ok deraadt@
2014-11-20 05:51:20 +00:00
doug 7fe19e9c57 Add gcc format attributes to more warn/error functions in parse.y files.
Fix a few missing or incorrect format characters.
ok claudio@
2014-11-03 03:42:11 +00:00
claudio 2fe39d326d Print bytes read and written in human readable form, like:
2849359 I2T calls (5 read, 2849352 writes)
5 data reads (2.3K bytes read)
2849352 data writes (43.5G bytes written)
2849358 T2I calls (2849358 done, 0 sense errors, 0 errors)
2014-04-21 20:20:37 +00:00
claudio 75b0049144 Document show summary 2014-04-21 18:05:45 +00:00
claudio 0ffafa029a Print the target and initiator name in show command. 2014-04-21 18:01:08 +00:00
claudio d9ff547311 Cope with the iscsid changes. Makes the msg handling a lot easier.
This also introduces some stuff that is work in progress.
2014-04-21 17:44:47 +00:00
claudio 7a1fddfcf6 It does not make sense to specify a port for the initiatoraddr. 2014-04-21 17:33:20 +00:00
jmc a2db36bbfc tweak previous; 2014-04-20 22:52:26 +00:00
claudio cf4672e7b9 Add iscsictl bits to show the vscsi stats. 2014-04-20 22:22:18 +00:00
benno 015d7b4d30 use u_char for buffers in yylex, for ctype calls
found by millert@, ok deraadt@
2013-11-25 12:51:10 +00:00
claudio a2460a9953 There is no need to compile this with -Werror by default. 2012-09-13 05:50:15 +00:00
haesbaert 9b9a50231f Define empty CDIAGFLAGS for programs that use Werror.
Makes "make build" build with WARNINGS=Yes on amd64.

ok espie
2012-09-12 09:19:54 +00:00
claudio cbe25abe77 Long overdue addition of the iscsi.conf man page.
Help and OK jmc@, OK dlg@
2012-06-26 21:09:44 +00:00
gsoares 761f0b5254 s/ospfd/iscsid/ in comment
sure claudio@
2012-05-02 18:02:45 +00:00
claudio 3471b91252 Implement "log verbose|brief" and allow the configuratuin of the
initiatior config which is just the ISID in case someone needs
to fix them to a specific value.
2011-04-27 19:20:01 +00:00
sobrado 7aed54e162 spacing, no binary changes. 2010-09-25 16:20:06 +00:00
dlg 211ccd835e basic manpage for iscsictl. we wont do an iscsi.conf manpage until the
syntax has settled down.
2010-09-24 12:31:35 +00:00
claudio 208af72238 Import iscsictl -- the control tool for iscsid.
Currently only discovery and reload work.
OK dlg@, matthew@, deraadt@
2010-09-24 09:45:17 +00:00