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.

Sign up for GitHub

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

Jump to bottom

test: move test_rtcp_decode_badmsg() to separate testcase #1041

Merged

sreimers merged 2 commits into main from test_rtcp_decode_badmsg

Jan 5, 2024

Contributor

alfredh commented Jan 4, 2024 •

edited

Loading

cppcheck:

test/rtp.c:424:9: warning: Identical condition and return expression 'err', return value is always 0 [identicalConditionAfterEarlyExit]
 return err;
        ^
test/rtp.c:417:6: note: If condition 'err' is true, the function will return/exit
 if (err)
     ^
test/rtp.c:424:9: note: Returning identical expression 'err'
 return err;
        ^

alfredh added 2 commits

January 4, 2024 10:56


          test: move test_rtcp_decode_badmsg() to separate testcase

cdffbd0


          test: fix test_rtcp_decode_badmsg for OOM

adc1e77

Contributor Author

alfredh commented Jan 5, 2024

the PR is complete and retest is also happy..

please review and merge ..

sreimers merged commit 34cce57 into main

44 checks passed

sreimers deleted the test_rtcp_decode_badmsg branch

January 5, 2024 19:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet