Page MenuHomeFreeBSD
Differential D55090

hdb_add_current_keys_to_history: pass correct `size` to calloc
AbandonedPublic
Actions

Authored by ngie on Tue, Feb 3, 8:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 18, 2:31 AM
Unknown Object (File)
Mon, Feb 16, 8:53 AM
Unknown Object (File)
Wed, Feb 11, 5:41 AM
Unknown Object (File)
Tue, Feb 10, 7:54 AM
Unknown Object (File)
Tue, Feb 10, 2:55 AM
Unknown Object (File)
Mon, Feb 9, 4:45 AM
Unknown Object (File)
Wed, Feb 4, 11:17 AM
Unknown Object (File)
Tue, Feb 3, 11:40 PM
View All 11 Files
Subscribers
imp
jhb

Details

Reviewers
rmacklem
cy
glebius
Summary

The size being passed to calloc prior to this change was incorrectly
calculated. The value passed should have been the type, not a
dereferenced pointer to a scalar value that happens to be the type.

This also avoids a NULL pointer dereference, which would have resulted
in a crash.

This fixes a -Wfree-nonheap-object issue cited by gcc 14.

MFC after: 3 days
PR: 292916

Test Plan

The code compiles/links without warnings with gcc14 on :main after this change.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70379
Build 67262: arc lint + arc unit

Event Timeline

ngie created this revision.Tue, Feb 3, 8:20 PM
Herald added a subscriber: imp. · View Herald TranscriptTue, Feb 3, 8:20 PM
ngie requested review of this revision.Tue, Feb 3, 8:20 PM
Harbormaster completed remote builds in B70379: Diff 171107.Tue, Feb 3, 8:20 PM
ngie edited the test plan for this revision. (Show Details)Tue, Feb 3, 8:26 PM
ngie added reviewers: rmacklem, cy.
ngie retitled this revision from hdb_add_current_keys_to_history: pass correct memory size to calloc to hdb_add_current_keys_to_history: pass correct `size` to calloc.
ngie added a subscriber: jhb.
ngie edited the summary of this revision. (Show Details)Tue, Feb 3, 8:29 PM
ngie added a reviewer: glebius.Wed, Feb 4, 8:11 AM
glebius added a comment.Wed, Feb 4, 5:09 PM

I can't understand the problem. If structure is known sizeof(*ext) results in the same value as sizeof(struct HDB_extension). If the structure is not known and has only forward declaration, compilation should fail with: invalid application of 'sizeof' to an incomplete type.

ngie abandoned this revision.Wed, Feb 4, 6:46 PM
In D55090#1259494, @glebius wrote:

I can't understand the problem. If structure is known sizeof(*ext) results in the same value as sizeof(struct HDB_extension). If the structure is not known and has only forward declaration, compilation should fail with: invalid application of 'sizeof' to an incomplete type.

Yeah, you're right. This change doesn't do anything difference net-wise (and TIL re: dereferencing points in this way). Abandoning...

Revision Contents
Changeset List

PathSize
crypto/
heimdal/
lib/
hdb/
2 lines

Diff 171107

View Options

crypto/heimdal/lib/hdb/keys.c

Loading...