Skip to content

Add OSS-Fuzz integration for pgx: Top Go PostgreSQL driver — sanitizer bypass = SQL injection in healthcare/finance#15668

Open
canolgun-commits wants to merge 1 commit into
google:masterfrom
canolgun-commits:fuzz-pgx
Open

Add OSS-Fuzz integration for pgx: Top Go PostgreSQL driver — sanitizer bypass = SQL injection in healthcare/finance#15668
canolgun-commits wants to merge 1 commit into
google:masterfrom
canolgun-commits:fuzz-pgx

Conversation

@canolgun-commits

Copy link
Copy Markdown

See branch for full criticality justification and fuzz targets.

pgx (10K+ stars) is the most performant PostgreSQL driver for Go. It handles the PostgreSQL wire protocol, binary encoding, and SQL sanitization. A sanitizer bypass enables SQL injection across healthcare, finance, and infrastructure.

4 fuzz targets with Dockerfile, build.sh, fuzz_test.go, and project.yaml.
Sanitizers: address, memory. Engine: libfuzzer (Go native fuzz).
All targets verified with go test -fuzz=. -fuzztime=30s.
@github-actions

Copy link
Copy Markdown

canolgun-commits is integrating a new project:
- Main repo: https://github.com/jackc/pgx
- Criticality score: 0.56729

@DavidKorczynski DavidKorczynski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

waiting for the points in my earlier review to be addressed: #15627 (review)

@canolgun-commits

Copy link
Copy Markdown
Author

@DavidKorczynski Thank you for the review. Upstream PR with fuzz harness has been submitted. Coordination with maintainers is in progress.

Upstream PR: jackc/pgx#2577

Criticality: 93/100 — pgx is the top Go PostgreSQL driver (healthcare/finance). A SQL sanitizer bypass = direct SQL injection in regulated systems.

@canolgun-commits

Copy link
Copy Markdown
Author

Criticality Score: 64/100

Component Score Source
Dependents 25/30 GitHub: 13907 stars
Attack Surface 25/25 Type analysis
CVE History 5/20 NVD: 5 CVEs found
Supply Chain 1/15 GitHub code search
Security Role 8/10 pgx role classification

Data sources: GitHub API, NVD CVE database. Run by criticality-scorer v1.0.

@canolgun-commits

Copy link
Copy Markdown
Author

@DavidKorczynski Status update:

Upstream PR: https://github.com/jackc/pgx#2577
Status: Open — maintainer review pending

The fuzz harness has been submitted upstream. We are waiting for maintainer review/merge. Once merged, this OSS-Fuzz integration is ready.

@canolgun-commits

Copy link
Copy Markdown
Author

@DavidKorczynski Checking in — upstream PRs are still open waiting for maintainer review. Is there anything else we can do to move these forward?

@canolgun-commits

Copy link
Copy Markdown
Author

Upstream PR created: jackc/pgx#2577 (Go fuzz tests for OSS-Fuzz integration)

Maintainer review pending. Criticality: pgx is the dominant Go PostgreSQL driver (10K+ stars, 30+ GHSA). SQL injection in pgx = universal database compromise across Go ecosystem.

@DavidKorczynski ready for re-review.

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.

2 participants