Skip to content

OSS-Fuzz: Add fuzzer targeting sip-dialog implementation#4848

Merged
sauwming merged 3 commits intopjsip:masterfrom
arthurscchan:sip-dialog
Mar 13, 2026
Merged

OSS-Fuzz: Add fuzzer targeting sip-dialog implementation#4848
sauwming merged 3 commits intopjsip:masterfrom
arthurscchan:sip-dialog

Conversation

@arthurscchan
Copy link
Contributor

This PR adds a new fuzzer targeting dialog handling implementation for SIP.

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
@arthurscchan arthurscchan marked this pull request as ready for review March 11, 2026 22:32
@sauwming sauwming requested a review from Copilot March 11, 2026 23:39
@sauwming sauwming added this to the release-2.17 milestone Mar 11, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an OSS-Fuzz-style harness to exercise SIP dialog creation/handling code paths, along with a seed corpus and build integration in the fuzz Makefile.

Changes:

  • Added a new fuzzer fuzz-sip-dialog that creates both UAC and UAS dialogs and drives basic dialog APIs.
  • Added a seed corpus zip for the new fuzzer.
  • Updated tests/fuzz/Makefile to build and clean the new fuzz target.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 4 comments.

File Description
tests/fuzz/seed/fuzz-sip-dialog_seed_corpus.zip Adds initial seed inputs for the new dialog fuzzer.
tests/fuzz/fuzz-sip-dialog.c New fuzz harness targeting SIP dialog APIs (UAC/UAS creation, route-set, request/response creation, session refs).
tests/fuzz/Makefile Builds and cleans the new fuzz-sip-dialog binary alongside existing fuzzers.

You can also share your feedback on Copilot code review. Take the survey.

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
@sauwming sauwming requested a review from nanangizz March 12, 2026 22:44
@sauwming sauwming merged commit 1446fce into pjsip:master Mar 13, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants