procd/utils, branch master

inittab: Disable implicit controlling TTY.

2024-12-11T11:09:18Z

Disable the use of implicit controlling TTYs. They will be enabled on demand. This fixes a bug where 2 or more consecutive Ctrl-C at the login prompt triggers a reboot of the device. Closes: https://github.com/openwrt/openwrt/issues/11306 Signed-off-by: Markus Gothe

utils: use strlcpy when appropriate

2024-07-07T17:00:35Z

For util functions called with a buffer and length parameter we should use strlcpy() instead of strncpy(), as those functions are called with sizeof(buffer) as parameter. Signed-off-by: Daniel Golle

utils: get_cmdline_val: add missing error check

2024-07-01T17:13:59Z

Add check to prevent read() in case open() has failed. Coverity CID: 1595922: Error handling issues (NEGATIVE_RETURNS) Fixes: 6f52ef3 ("get_cmdline_val: search for entire name, not just suffix") Signed-off-by: Daniel Golle

utils: add key-value offset support to get_cmdline_val()

2024-03-25T09:28:50Z

Add offset support of key-value pairs to get_cmdline_val() for getting multiple values of the same keys. Signed-off-by: INAGAKI Hiroshi

utils: don't ignore open() return value

2021-08-30T23:46:49Z

In case active console cannot be opened, return NULL early instead of trying to read from errornous file descriptor. Coverity CID: 1490087 Argument cannot be negative Signed-off-by: Daniel Golle

utils: make sure read() string is 0 terminated

2021-08-24T17:31:46Z

Coverity CID: 1490062 String not null terminated Signed-off-by: Daniel Golle

inittab: detect active console from kernel if no console= specified

2021-03-07T23:46:27Z

The default serial console can be set in the device tree using the linux,stdout-path parameter (or equivalent from ACPI). This is important for universal booting (EFI/EBBR) on ARM platforms where the default console can be different (e.g ttyS0 vs ttyAMA0). Signed-off-by: Mathew McBride

utils: fix C style in header file

2021-03-07T23:45:33Z

Signed-off-by: Daniel Golle

procd: check strchr() result before using it

2019-07-13T07:22:52Z

Subtracting some address from NULL does not necessary results in negative value. It's lower level dependent. In our case (IPQ4019 + Yocto + meta-openwrt) subtracting token address from NULL strchr() return value results in large positive number which causes out-of-boundary memory access and eventually a segfault. Signed-off-by: Justinas Grauslis

procd: Remove redundant errno variable in several printf functions.

2018-01-02T12:01:31Z

%m, aka strerror(errno) is already used. No need to repeat it. Saves 280 bytes. Signed-off-by: Rosen Penev