Commits


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.


avoid use of uninitialized variable 'sin' found by codechecker


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


ifmedia words are 64 bit wide on openbsd found by codechecker


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


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


need to free vis on error in mbs2ws() found by codechecker


plug file descriptor leak in who.c found by codechecker


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


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


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


preserve original pointer during realloc and free on error found by codechecker


preserve original pointer during realloc() and free on error return found by codechecker


Update README.md improve formatting and spacing of NSH docs


Update README.md with latest manual and recent youtube videos on NSH


Update README.md Update with rendering from latest manual page


allow the interface command to work within an interface context ok Tom


Merge pull request #154 from smytht/exclamation-mark-change ! without arguments should not fork a new shell


reword an informative warning message such that it sounds less like an error


Update commands.c fix typo in exit command in bridge commands struct


fix typo in exit command in bridge commands struct This is the intended change from commit cfd60254


! without arguments should not fork a new shell This Diff changes the behaviour of nsh interactively by checking if ! has been entered without any arguments, if the user is in global config mode or privileged mode the user is returned to the same prompt. if the user is in interface config mode or bridge config mode the user is returned to the global configuration mode This is to facilitate users copying and pasting nshrc configuration from one device to another interactively.


backout: fix typo in bridge commands struct (missing) field for exit command Unintended changes were mixed into this commit; discussed with Tom


Merge pull request #153 from stspdotname/pairfix fix conf_patch to avoid overwriting the current interface being shown I have reviewed the Code what stsp describes makes sense and the diff works and appears to solves the issue raised by ryan-sg OK Tom


fix typo in bridge commands struct (missing) field for exit command fix typo in bridge commands struct (missing) field for exit command