Skip to content

ci: add a GHAction sending data to Coverity#4

Merged
evverx merged 1 commit intomasterfrom
coverity
Nov 3, 2021
Merged

ci: add a GHAction sending data to Coverity#4
evverx merged 1 commit intomasterfrom
coverity

Conversation

@evverx
Copy link
Owner

@evverx evverx commented Nov 3, 2021

No description provided.

@evverx evverx force-pushed the coverity branch 9 times, most recently from 4216b5a to e85ef48 Compare November 3, 2021 21:27
@evverx evverx marked this pull request as ready for review November 3, 2021 21:28
@evverx evverx merged commit 3c632b7 into master Nov 3, 2021
evverx pushed a commit that referenced this pull request Nov 18, 2022
This option affects list of visible columns, so it's better to apply
it before any other flags, because "-1" placeholder depends on visible
columns. For example:

	echo "A B C D" | column --table --table-maxout --table-right -1 --table-hide 4

needs to hide column #4 (D) and align #3 (C) to the right.

Signed-off-by: Karel Zak <[email protected]>
evverx pushed a commit that referenced this pull request Apr 29, 2023
If the fmt being parsed by block_size is exactly '%', *++fmt is the
terminator and strchr will return a valid pointer to the terminator of
spec rather than NULL, the while condition will pass and subsequent
strchr will read past the end of fmt until a spec character is found
again

ASAN aborts with the following error on the first buffer overread:

AddressSanitizer: heap-buffer-overflow on address 0x602000000212 at pc 0x55bf1c4b2d78 bp 0x7ffe33c8cff0 sp 0x7ffe33c8cfe0
READ of size 1 at 0x602000000212 thread T0
    #0 0x55bf1c4b2d77 in block_size ../text-utils/hexdump-parse.c:207
    #1 0x55bf1c4ad36f in main ../text-utils/hexdump.c:214
    #2 0x7f15f063c78f  (/usr/lib/libc.so.6+0x2378f)
    #3 0x7f15f063c849 in __libc_start_main (/usr/lib/libc.so.6+0x23849)
    #4 0x55bf1c4ac6c4 in _start (../build/hexdump+0x86c4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant