pkg: Fix Coverity warnings
ClosedPublic
Actions

Authored by markj on Jan 16 2025, 3:12 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Aug 13, 4:55 AM

	Unknown Object (File)
	Thu, Aug 13, 4:25 AM

	Unknown Object (File)
	Thu, Aug 13, 2:16 AM

	Unknown Object (File)
	Sat, Aug 8, 4:59 PM

	Unknown Object (File)
	Fri, Aug 7, 7:33 PM

	Unknown Object (File)
	Thu, Aug 6, 1:05 AM

	Unknown Object (File)
	Wed, Aug 5, 6:17 AM

	Unknown Object (File)
	Wed, Aug 5, 2:25 AM

View All 82 Files

Subscribers

imp

Details

Reviewers

bapt
bdrewery

Summary

Fix allocation size in config_get_repositories().
Fix a memory leak in read_conf_file().
Avoid a null pointer dereference in an error path in verify_pubsignature().

Fixes: e3b4a51580fc ("pkg(7): expand VERSION_MAJOR, VERSION_MINOR, RELEASE and OSNAME")
Fixes: dc4581589a32 ("pkg: clean support for repositories")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 61765
Build 58649: arc lint + arc unit

Event Timeline

markj created this revision.Jan 16 2025, 3:12 PM

Herald added a reviewer: bdrewery. · View Herald TranscriptJan 16 2025, 3:12 PM

Herald added a subscriber: imp. · View Herald Transcript

markj requested review of this revision.Jan 16 2025, 3:12 PM

Harbormaster completed remote builds in B61765: Diff 149343.Jan 16 2025, 3:12 PM

bapt accepted this revision.Jan 16 2025, 3:39 PM

This revision is now accepted and ready to land.Jan 16 2025, 3:39 PM

Committed in https://cgit.freebsd.org/src/commit/?id=c1557708f1fae1bb9c8e23e3bbb2aa2b055e1211, I forgot to tag the review.

Revision Contents
Changeset List

Path

Size

usr.sbin/

pkg/

config.c

14 lines

pkg.c

2 lines

Diff 149343

View Options

usr.sbin/pkg/config.c