Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ABORT THROWN BY INTERNAL EXCEPTION: Assertion triggered in file "/home/runner/work/duckdb-fuzzer-ci/duckdb-fuzzer-ci/src/common/types/hugeint.cpp" on line 129: lhs.upper >= 0 ../duckdb(+0x18efd814) [0x558aae153814] ../duckdb(+0x18dd1d3d) [... #3990

Closed
fuzzerofducks opened this issue Feb 13, 2025 · 1 comment · Fixed by duckdb/duckdb#16394
Assignees

Comments

@fuzzerofducks
Copy link
Collaborator

Issue found by DuckFuzz (Functions) on git commit hash using seed 625199353.

To Reproduce

call dbgen(sf=0.1);
SELECT mean(c1) FROM test_vector_types(CAST(NULL AS TIMESTAMP WITH TIME ZONE)) AS test_vector_types(c1);

Error Message

ABORT THROWN BY INTERNAL EXCEPTION: Assertion triggered in file "/home/runner/work/duckdb-fuzzer-ci/duckdb-fuzzer-ci/src/common/types/hugeint.cpp" on line 129: lhs.upper >= 0

../duckdb(+0x18efd814) [0x558aae153814]
../duckdb(+0x18dd1d3d) [0x558aae027d3d]
../duckdb(+0x18f671c6) [0x558aae1bd1c6]
../duckdb(+0x18d98c68) [0x558aadfeec68]
../duckdb(+0x186dcf83) [0x558aad932f83]
../duckdb(+0x20d84311) [0x558ab5fda311]
../duckdb(+0x20d5cab4) [0x558ab5fb2ab4]
../duckdb(+0x20d4cd61) [0x558ab5fa2d61]
../duckdb(+0x1d70fb33) [0x558ab2965b33]
../duckdb(+0x1d71020b) [0x558ab296620b]
../duckdb(+0x1a0cfc5c) [0x558aaf325c5c]
../duckdb(+0x1a0d0437) [0x558aaf326437]
../duckdb(+0x1a0ca39c) [0x558aaf32039c]
../duckdb(+0x1a0b4268) [0x558aaf30a268]
../duckdb(+0x1a098283) [0x558aaf2ee283]
../duckdb(+0x1a0aab3c) [0x558aaf300b3c]
../duckdb(+0x19b42c43) [0x558aaed98c43]
../duckdb(+0x19bb5a33) [0x558aaee0ba33]
../duckdb(+0x19bb5fb9) [0x558aaee0bfb9]
../duckdb(+0x19bb68ef) [0x558aaee0c8ef]
../duckdb(+0x17da5af3) [0x558aacffbaf3]
../duckdb(+0x17d1f66a) [0x558aacf7566a]
../duckdb(+0x17d210ad) [0x558aacf770ad]
../duckdb(+0x17d3f165) [0x558aacf95165]
../duckdb(+0x17d402ae) [0x558aacf962ae]
../duckdb(+0x17d44c90) [0x558aacf9ac90]
/lib/x86_64-linux-gnu/libc.so.6(+0x2a1ca) [0x7f92ca82a1ca]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x8b) [0x7f92ca82a28b]
../duckdb(+0x17ca3025) [0x558aacef9025]
@Tmonster
Copy link

Seems like 1e404813c9ec1ebe47e4d1b4dccfd86ad978ba7d is the commit that has caused this issue

@Tmonster Tmonster self-assigned this Feb 14, 2025
This was referenced Feb 21, 2025
This was referenced Feb 24, 2025
Mytherin added a commit to duckdb/duckdb that referenced this issue Feb 26, 2025
Fixes duckdb/duckdb-fuzzer#3990

Fix (I think) Is to not use the DivModPositive Function as it assumes
the LHS is positive. However, it seems like when NULL is cast to
`Timestamp` a negative timestamp/hugeint_t value is used.

Another fix may be a different timestamp value when casting NULL.

Commit causing the error was
[1e40481](1e40481).

CC @hawkfish
This was referenced Feb 28, 2025
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 a pull request may close this issue.

2 participants