Differential D48484

pkg: Fix Coverity warnings
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F109247387: D48484.diff
	Sun, Feb 2, 2:09 PM

	Unknown Object (File)
	Wed, Jan 29, 9:04 AM

	Unknown Object (File)
	Sun, Jan 26, 10:09 AM

	Unknown Object (File)
	Wed, Jan 22, 5:18 PM

	Unknown Object (File)
	Mon, Jan 20, 3:11 AM

	Unknown Object (File)
	Sun, Jan 19, 7:08 PM

	Unknown Object (File)
	Sun, Jan 19, 12:11 PM

Subscribers

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.Thu, Jan 16, 3:12 PM

Herald added a reviewer: bdrewery. · View Herald TranscriptThu, Jan 16, 3:12 PM

Herald added a subscriber: imp. · View Herald Transcript

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

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

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

This revision is now accepted and ready to land.Thu, Jan 16, 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/

14 lines

2 lines

Diff 149343

usr.sbin/pkg/config.c

Loading...

usr.sbin/pkg/pkg.c

Loading...