Age | Commit message (Collapse) | Author |
|
* check strdup for malloc failure
* remove obvious /* NOTREACHED */
* return instead of exit from main
* err(1, NULL) instead of err(1, "malloc")
* mark usage as __dead
ok deraadt
|
|
|
|
rather than opening a fresh socket on the fly.
ok krw
|
|
upon "inet". Adjust the 4 programs that care about this.
|
|
ok deraadt@
|
|
|
|
Restore gcc's equanimity by tedu@'ing the prototype.
|
|
"warning:" rectification.
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
From and OK markus@, OK reyk
|
|
bfd session details are visible with "route -n get 192.0.2.1 -bfd"
OK mpi@ deraadt@ claudio@
|
|
steal code.
|
|
ok deraadt@ millert@ mpi@
|
|
This replaces log_verbose() and "extern int verbose" with the two functions
log_setverbose() and log_getverbose().
Pointed out by benno@
OK krw@ eric@ gilles@ (OK gilles@ for the snmpd bits as well)
|
|
TAILQ_FOREACH().
No intentional functional change.
ok reyk@
|
|
so it is safe calling log_* after an error without loosing the it.
|
|
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@
|
|
Brought up by doug@, ok reyk, djm, doug
|
|
As a related change, load the local.pub and local.key keys after
privsep and reload them on SIGHUP/reload.
OK mikeb@
|
|
if they precede the noun and omit hyphens otherwise.
ok tj
|
|
often confusing than useful these days. Stop emitting them.
Prompted by bugs@ report of the 8TB disk travails of Jiri.
ok deraadt@ tom@
|
|
so gcc doesn't complain about defining but not using them.
typo correction & ok jca@
|
|
ok tom@
|
|
ok mpi@
|
|
bio status will never be updated on a BIOCLOCATE. In addition with
missed zeroing of the passed bio struct, this could lead to a print
of uninitialized memory. While there, properly zero the bio struct
before passing it to ioctl().
ok mikeb@
|
|
|
|
include the wpakey) to their defaults.
And make 'ifconfig if0 wpaprotos' reset WPA crypto parameters to settings
which are appropriate for the specified WPA protocol version.
|
|
adding appropriate #ifdef's around declarations.
ok millert@ (with a tweak I will commit separately)
|
|
Issue reported by jsing@, ok stsp@
|
|
setting a tunnel addresses uses a sockaddr_in6 from getaddrinfo, which
sets sin6_scope_id for scoped addresses. this is nice and portable.
reading a tunnel address assumed the scope was embedded in link local
addresses in sin6_addr, and unpacked the scope back into sin6_scope_id.
this is inconsistent with the code that sets tunnel addresses.
this fixes the read path so it assumes the kernel sets sin6_scope_id
like it it assumes the kernel will read on the set side. noone likes
the kame hack of embedding the scope id in the addreses.
ok mpi@
|
|
diff from jerome frgacic, tweaked by myself
ok krw millert
|
|
|
|
|
|
dedicated number. Both changes for consistency.
OK rzalamena@
|
|
ok tb@ danj@ deraadt@
|
|
ok mikeb@
|
|
lease when DHO_CLASSLESS_[MS_]STATIC_ROUTES are present. RFC 3442 says
the client must ignore the former if the latter is present.
The information was already ignored but was still written to the
'effective' section of the pseudo-lease file created by the -L
option. This incorrectly implied that these options were 'effective'.
Problem noted by Ed Fochler. Thanks!
|
|
parser cannot handle that correctly and is is unclear wether the
kernel code would work. Remove the feature until someone needs it
and properly implements and tests it.
OK mike@ sashan@ mpi@
|
|
same keys as the actual commands (datapath, maxgroup, flowmax).
flowmax should eventually renamed to maxflow for consistency.
OK rzalamena@
|
|
of the output buffer size. Thus elements like subscriber-id, ICC-id, IMEI
were shown truncated.
Some modules report a phone number that already has the '+' prefix.
Don't add another one when printing it.
Patch from Bryan Vyhmeister
ok otto
|
|
static, so keep it that way for consistency)
- s/usage(1)/usage() and inside call exit(1) explicitly since all usage() calls
always use that value (also update comment to reflect this change)
- Remove main() prototype
- s/exit/return in main() to enable SSP
Feedback from jca@ and tb@ and OK from both (with their remarks in)
|
|
"twenty four", arguing, i think, that the internet is wrong; well
seemingly i'm wrong, and you should hyphenate numbers when spelled out
- who'd have thought it. apologies to florian and the diff submitter
(and the internet);
still, what i should have advised, and what i've done here, is use "24",
which is much simpler, fits the surrounding text, and agrees with oed
style advice;
|