Commit Briefs

f59823bb56 GitHub

Version Bump for release 1.2.2 (tags/v1.2.2)



741b54eca9 GitHub

Update README.md ahead of release


1bcd050e14 GitHub

minor corrections to nsh.8 before release

mandoc -Tlint



21b1dfdbf5 Stefan Sperling

add a 'make static' target which builds statically linked binaries

The goal is to make it easier to build the -static flavour of nsh in the OpenBSD ports tree.


44b5f33be4 GitHub

su is an alias of enable for convenience

su should not be displayed in help / context help in nsh su will work but not be suggested to user during tab completion



ae3b5241d9 GitHub

Update extensive-nsh-openbsd-integrate.sh

remove hostname.if and mygate on Integration script run per comment above those lines and add commands to backup the hostname and hosts file configuration


65acc1ab45 Stefan Sperling

fix adding blackhole and reject routes; a gateway is required

Fixes an invalid route message error during nsh -i at boot time: % cmd: route (line 49) margv [route 224.0.0.0/4 127.0.0.1 iface nompath reject] % RTM_ADD: Add Route: len 128, table 0, pid: 0, seq 1, errno 0, flags:<UP,REJECT,STATIC> % locks: inits: % sockaddrs: <DST,NETMASK> 224.0.0.0 240.0.0.0 % ip_route: rtmsg: Invalid argument test + ok Tom


1ecf319919 Stefan Sperling

create save.sh during "make install" if it is missing


45a20fd042 Stefan Sperling

fix whitespace




da2af3ac49 Stefan Sperling

silence ifconfig errors related to carp demotion at boot time

This happens when nsh is started by /etc/rc.d/nsh during boot. The /etc/rc script starts package scripts with carp-interlock already enabled, and both ifconfig and nsh would attempt to reset the carp demotion counter. Generally, this is harmless, apart from an error message displayed by ifconfig which might mislead users into assuming there was a problem: starting package daemons: nsh. ifconfig: carp: SIOCSIFGATTR: Invalid argument To avoid this issue, nsh can check whether the demotion counter has already been set to the desired level and only change the counter if changing it is indeed required. test + ok Tom


ddce9e236b Stefan Sperling

avoid use of uninitialized variable 'sin'

found by codechecker


76ca437888 Stefan Sperling

call getprogname(3) instead of reading __progname which is reserved

found by codechecker


0afff63a21 Stefan Sperling

ifmedia words are 64 bit wide on openbsd

found by codechecker


84d6fbcbec Stefan Sperling

handle SIGWINCH in a signal-safe way

The setwinsize() function calls an ioctl which is not async-signal-safe. Use a safe signal handler and change window size in command loop instead. found by codechecker


3ce23c08d0 Stefan Sperling

left-shift by 32 bits needs to use unsigned long long

found by codechecker


ed18441760 Stefan Sperling

need to free vis on error in mbs2ws()

found by codechecker


c550e428fb Stefan Sperling

plug file descriptor leak in who.c

found by codechecker


50e10c0364 Stefan Sperling

handle edge-case of zero no-commands in complete_nocmd(); should not happen

found by codechecker


7988c7a7ca Stefan Sperling

must initialize ndp.c variable 'nbi' in case for-loop never runs

found by codechecker


cba0363734 Stefan Sperling

in growwgdata, preserve original pointer during realloc for freeing on error

found by codechecker