Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2025-09-08 | snmp: fix complaint of snmpd about paths | Lorenz Kästle | 1 | -1/+1 | |
2025-09-01 | lib/utils_base.c: small refactoring | Lorenz Kästle | 2 | -103/+94 | |
2025-09-01 | lib/utils_base.c: clang-format | Lorenz Kästle | 1 | -13/+18 | |
2025-09-01 | check_snmp: improve string quoting in result | Lorenz Kästle | 1 | -1/+16 | |
2025-08-30 | check_snmp: Small improvements + fix dereference bug | Lorenz Kästle | 1 | -3/+4 | |
2025-08-30 | check_snmp: Fix number processing (offset + multiplier) | Lorenz Kästle | 3 | -13/+62 | |
2025-08-30 | check_snmp: implement output format setting | Lorenz Kästle | 2 | -15/+46 | |
2025-08-29 | check_snmp: Make linter happy | Lorenz Kästle | 1 | -22/+28 | |
2025-08-29 | check_snmp: Remove options description for input delimiter | Lorenz Kästle | 1 | -6/+0 | |
2025-08-28 | Build check_snmp unconditionally | Lorenz Kästle | 1 | -2/+3 | |
2025-08-28 | check_snmp: fix DES availability detection(?) | Lorenz Kästle | 2 | -3/+13 | |
2025-08-28 | check_snmp: formatting | Lorenz Kästle | 1 | -2/+2 | |
2025-08-28 | check_snmp: Test for availability of DES privacy protocol | Lorenz Kästle | 1 | -3/+18 | |
2025-08-28 | Remove testing for SNMPGETNEXT from autotools stuff | Lorenz Kästle | 1 | -11/+0 | |
2025-08-27 | check_snmp: use snmp v2c if community is given | Lorenz Kästle | 1 | -0/+8 | |
2025-08-27 | Fix Specfile requires | Lorenz Kästle | 1 | -1/+3 | |
2025-08-27 | Add libsnmp-dev dependency to codeql gh action | Lorenz Kästle | 1 | -3/+14 | |
2025-08-27 | Fix one more typo | Lorenz Kästle | 1 | -1/+1 | |
2025-08-27 | Add netsnmp lib to specfile | Lorenz Kästle | 1 | -0/+1 | |
2025-08-27 | Add libsnmp-dev to github action dependencies | Lorenz Kästle | 1 | -0/+1 | |
2025-08-27 | check_snmp: fix typos | Lorenz Kästle | 1 | -6/+6 | |
2025-08-27 | check_snmp: declare internal functions static | Lorenz Kästle | 1 | -2/+2 | |
2025-08-27 | check_snmp: remove leftover multiply function | Lorenz Kästle | 1 | -48/+0 | |
2025-08-27 | check_snmp: fix/adapt tests | Lorenz Kästle | 1 | -62/+41 | |
2025-08-27 | check_snmp: hopefully fix helpers | Lorenz Kästle | 2 | -32/+41 | |
2025-08-27 | check_snmp: Improve error handling | Lorenz Kästle | 1 | -6/+15 | |
2025-08-27 | check_snmp: rebuild threshold parsing | Lorenz Kästle | 5 | -65/+125 | |
2025-08-27 | Fix range comparison and aesthetic improvements | Lorenz Kästle | 1 | -5/+5 | |
2025-08-25 | rebuild check_snmp | Lorenz Kästle | 3 | -926/+838 | |
2025-08-11 | Merge pull request #2107 from RincewindsHat/refactor/check_users | Lorenz Kästle | 7 | -123/+343 | |
Refactor/check users | |||||
2025-08-11 | Merge branch 'master' into refactor/check_users | Lorenz Kästle | 68 | -5842/+6408 | |
2025-08-11 | check_users: Use sd_get_uids instead of sd_get_session | Lorenz Kästle | 1 | -1/+1 | |
Previously check_users in combination with systemd used sd_get_sessions (3) to aquire the number of users, probably with the idea that every users opens a session. Turns out, that a user can have multiple sessions and we only really want to know how many users there are. This commit changes to sd_get_uids (3) to achieve that target. | |||||
2025-08-04 | Merge pull request #2139 from waja/ci_fix_rawhide | waja | 1 | -0/+3 | |
2025-08-01 | (Re)construct PLATFORM_ID as it's droped since Fedora 43 | Jan Wagner | 1 | -0/+3 | |
See https://fedoraproject.org/wiki/Changes/Drop_PLATFORM_ID?#Drop_PLATFORM_ID | |||||
2025-08-01 | Merge pull request #2138 from RincewindsHat/improvements/check_ssh | Lorenz Kästle | 1 | -34/+47 | |
Improvements/check ssh | |||||
2025-08-01 | clang-format | Lorenz Kästle | 1 | -29/+43 | |
2025-08-01 | check_ssh: Put variable in the correct scope | Lorenz Kästle | 1 | -2/+1 | |
2025-08-01 | check_ssh: Fix format expression | Lorenz Kästle | 1 | -1/+1 | |
2025-08-01 | check_ssh: fix data type to allow for error checking | Lorenz Kästle | 1 | -2/+2 | |
2025-08-01 | Merge pull request #2133 from rlaager/fix-check_ssh-buffer-overflow | Lorenz Kästle | 1 | -4/+6 | |
Fix check ssh buffer overflow | |||||
2025-07-24 | Merge pull request #2137 from waja/ci_dispatch_ssh | waja | 2 | -2/+26 | |
Adding tmate optional to manual dispatch | |||||
2025-07-24 | Adding tmate optional to manual dispatch | Jan Wagner | 2 | -2/+26 | |
2025-07-23 | Merge pull request #2136 from waja/ci_dispatch | waja | 3 | -0/+3 | |
2025-07-23 | CI: Adding workflow_dispatch | Jan Wagner | 3 | -0/+3 | |
2025-07-14 | Merge pull request #2134 from RincewindsHat/fix/check_ntp_peer_fmt_conv | Lorenz Kästle | 1 | -3/+3 | |
check_ntp_peer: fix invalid conversion in printf | |||||
2025-07-14 | check_ntp_peer: fix invalid conversion in printf | Lorenz Kästle | 1 | -3/+3 | |
2025-07-11 | check_ssh: Correct type on len variable | Richard Laager | 1 | -1/+1 | |
strlen() returns a size_t. Signed-off-by: Richard Laager <rlaager@wiktel.com> | |||||
2025-07-11 | check_ssh: Fix buffer overflow | Richard Laager | 1 | -3/+5 | |
A buffer overflow was occurring when the server responded with: Exceeded MaxStartups\r\n glibc would then abort() with the following output: *** buffer overflow detected ***: terminated It was the memset() that was overflowing the buffer. But the memmove() needed fixing too. First off, there was an off-by-one error in both the memmove() and memset(). byte_offset was already set to the start of the data _past_ the newline (i.e. len + 1). For the memmove(), incrementing that by 1 again lost the first character of the additional output. For the memset(), this causes a buffer overflow. Second, the memset() has multiple issues. The comment claims that it was NULing (sic "null") the "rest". However, it has no idea how long the "rest" is, at this point. It was NULing BUFF_SZ - byte_offset + 1. After fixing the off-by-one / buffer overflow, it would be NULing BUFF_SZ - byte_offset. But that doesn't make any sense. The length of the first line has no relation to the length of the second line. For a quick-and-dirty test, add something like this just inside the while loop: memcpy(output, "Exceeded MaxStartups\r\nnext blah1 blah2 blah3 blah4\0", sizeof("Exceeded MaxStartups\r\nnext blah1 blah2 blah3 blah4\0")); And, after the memmove(), add: printf("output='%s'\n", output); If you fix the memset() buffer overflow, it will output: output='ext blah1 blah2 blah3 ' As you can see, the first character is lost. If you then fix the memmove(), it will output: output='next blah1 blah2 blah3' Note that this is still losing the "blah4". After moving the memset() after byte_offset is set to the new strlen() of output, then it works correctly: output='next blah1 blah2 blah3 blah4' Signed-off-by: Richard Laager <rlaager@wiktel.com> | |||||
2025-07-06 | Merge pull request #2109 from RincewindsHat/refactor/check_load | Lorenz Kästle | 5 | -294/+444 | |
Refactor/check load | |||||
2025-07-06 | check_load: fix tests | Lorenz Kästle | 1 | -3/+3 | |