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

stream: declare ext_len when assigned #2678

Merged
merged 1 commit into from
Aug 11, 2023
Merged

Conversation

alfredh
Copy link
Collaborator

@alfredh alfredh commented Aug 10, 2023

No description provided.

@alfredh
Copy link
Collaborator Author

alfredh commented Aug 10, 2023

Assertion failed: 0 && "RE_VA_ARG: arg is not compatible", file /home/runner/work/baresip/baresip/baresip-win32/re/src/fmt/print.c, line 348

@sreimers
Copy link
Member

Yes, that's strange, will try to investigate.

@alfredh
Copy link
Collaborator Author

alfredh commented Aug 11, 2023

HEAD is also failing with the same error:

Assertion failed: 0 && "RE_VA_ARG: arg is not compatible", file /home/runner/work/baresip/baresip/baresip-win32/re/src/fmt/print.c, line 348

It could be related to 32-bit platforms.

Is it possible to add some more detailed warnings or a backtrace ?

@sreimers
Copy link
Member

Yes it is related to the secure mbuf_printf merge. The failing test is test_ua_refer. It should generate a ci backtrace already but looks something is broken too. Try to debug locally.

@sreimers
Copy link
Member

Ok should be fixed by: baresip/re#903

@alfredh
Copy link
Collaborator Author

alfredh commented Aug 11, 2023

thanks for investigating and fixing

@alfredh alfredh merged commit 343f4b6 into main Aug 11, 2023
18 checks passed
@alfredh alfredh deleted the alfredh_stream_extlen_declare branch August 11, 2023 09:31
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.

None yet

2 participants