Page MenuHomeFreeBSD

hyperv: Fix Coverity issues in the Hyper-V driver
ClosedPublic

Authored by dab on Feb 12 2023, 11:21 PM.
Tags
None
Referenced Files
F98712363: D38551.diff
Fri, Oct 4, 11:05 AM
Unknown Object (File)
Wed, Oct 2, 2:16 AM
Unknown Object (File)
Sat, Sep 21, 2:19 AM
Unknown Object (File)
Thu, Sep 19, 3:22 AM
Unknown Object (File)
Wed, Sep 18, 7:06 PM
Unknown Object (File)
Tue, Sep 17, 9:08 PM
Unknown Object (File)
Tue, Sep 17, 12:57 AM
Unknown Object (File)
Sep 4 2024, 7:50 AM
Subscribers

Details

Summary

Coverity reports the use of two uninitialized variables in the hyperv
driver. Examination shows the variables can be accessed while
uninitialized in error cases. Fix both.

CID: 1365235
CID: 1365236
Obtained from: Robert Herndon <Robert.Herndon@Dell.com>
Sponsored by: Dell Technologies
MFC after: 1w

Test Plan

Changes in use at $WORK

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 49722
Build 46612: arc lint + arc unit

Event Timeline

dab requested review of this revision.Feb 12 2023, 11:21 PM

Common practice for attributing the submitter with git is:

git commit --amend --author='Robert Herndon <Robert.Herndon@Dell.com>'

This revision is now accepted and ready to land.Feb 14 2023, 2:55 PM

Common practice for attributing the submitter with git is:

git commit --amend --author='Robert Herndon <Robert.Herndon@Dell.com>'

Ah, good to know. I actually used git am to apply the patch and it annotated the commit that way. I "fixed" it to list me. I'll change it back and remember this for the future.

Does that then take the place of "Obtained from" and/or "Submitted by"?

Re-trying to approve this change. (Please ignore if redundant.)

Correct author in commit log.

This revision now requires review to proceed.Feb 14 2023, 4:51 PM
This revision is now accepted and ready to land.Feb 14 2023, 4:55 PM