Page MenuHomeFreeBSD
Differential D26037

Add test to verify fix for FreeBSD-SA-19:20.bsnmp
ClosedPublic
Actions

Authored by darrick.freebsd_gmail.com on Aug 12 2020, 12:18 AM.
Tags
None
Referenced Files
F145484346: D26037.id75711.diff
Fri, Feb 20, 10:23 AM
F145439449: D26037.id75806.diff
Thu, Feb 19, 10:17 PM
F145423232: D26037.id75712.diff
Thu, Feb 19, 4:21 PM
Unknown Object (File)
Tue, Feb 17, 5:19 PM
Unknown Object (File)
Thu, Jan 22, 10:26 PM
Unknown Object (File)
Jan 5 2026, 11:11 PM
Unknown Object (File)
Jan 5 2026, 2:00 AM
Unknown Object (File)
Jan 2 2026, 8:22 PM
View All Files
Subscribers
imp

Details

Reviewers
cem
vangyzen
Group Reviewers
tests
Commits
rS364221: Add test to for FreeBSD-SA-19:20.bsnmp
Summary

Verify passing in improper length in asn_buf to asn_get_header() returns ASN_ERR_EOBUF.

Test Plan

Verified test passing with fix for FreeBSD-SA-19:20.bsnmp and failing without.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

darrick.freebsd_gmail.com requested review of this revision.Aug 12 2020, 12:18 AM
darrick.freebsd_gmail.com created this revision.
Harbormaster completed remote builds in B32934: Diff 75711.Aug 12 2020, 12:18 AM
cem added a reviewer: tests.Aug 12 2020, 12:23 AM
cem added inline comments.
lib/libbsnmp/tests/bsnmpd_test.c
9 ↗(On Diff #75711)

The explicit length can be dropped to size it correctly automatically, char test_buf[] = { ... };

15 ↗(On Diff #75711)

2, or sizeof(test_buf)

darrick.freebsd_gmail.com updated this revision to Diff 75712.Aug 12 2020, 12:32 AM

Update per CR feedback.

Harbormaster completed remote builds in B32935: Diff 75712.Aug 12 2020, 12:32 AM
cem added inline comments.Aug 12 2020, 12:45 AM
lib/libbsnmp/tests/bsnmpd_test.c
15 ↗(On Diff #75711)

this one was missed?

darrick.freebsd_gmail.com updated this revision to Diff 75713.Aug 12 2020, 1:31 AM

Update per CR feedback.

Harbormaster completed remote builds in B32936: Diff 75713.Aug 12 2020, 1:31 AM
cem accepted this revision.Aug 12 2020, 8:44 PM
This revision is now accepted and ready to land.Aug 12 2020, 8:44 PM
Closed by commit rS364221: Add test to for FreeBSD-SA-19:20.bsnmp (authored by bdrewery). · Explain WhyAug 13 2020, 10:42 PM
This revision was automatically updated to reflect the committed changes.
bdrewery added a commit: rS364221: Add test to for FreeBSD-SA-19:20.bsnmp.
Herald added a subscriber: imp. · View Herald TranscriptAug 13 2020, 10:42 PM

Revision Contents
Changeset List

PathSize
head/
lib/
libbsnmp/
1 line
tests/
11 lines
53 lines

Diff 75806

View Options

head/lib/libbsnmp/Makefile

Loading...
View Options

head/lib/libbsnmp/tests/Makefile

Loading...
View Options

head/lib/libbsnmp/tests/bsnmpd_test.c

Loading...