Page MenuHomeFreeBSD
Differential D2342

Missing break.
ClosedPublic
Actions

Authored by araujo on Apr 22 2015, 8:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 4:11 AM
Unknown Object (File)
Wed, Dec 18, 3:21 AM
Unknown Object (File)
Sep 18 2024, 5:59 PM
Unknown Object (File)
Sep 5 2024, 9:21 PM
Unknown Object (File)
Sep 2 2024, 11:49 AM
Unknown Object (File)
Aug 31 2024, 4:43 PM
Unknown Object (File)
Aug 19 2024, 10:21 AM
Unknown Object (File)
Aug 10 2024, 9:47 AM
View All 53 Files
Subscribers
imp

Details

Reviewers
neel
Summary

Hi,

Is it missing a break there? Or it is intentional to fall throught to the next case?

It was reported by coverity CID: 1292499

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

araujo updated this revision to Diff 4946.Apr 22 2015, 8:54 AM
araujo retitled this revision from to Missing break..
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo added a subscriber: neel.
Herald added a subscriber: imp. · View Herald TranscriptApr 22 2015, 8:54 AM
araujo added a reviewer: neel.Apr 22 2015, 8:54 AM
araujo removed a subscriber: neel.
neel accepted this revision.Apr 22 2015, 5:21 PM
neel edited edge metadata.

Looks good. Thanks for catching this.

This revision is now accepted and ready to land.Apr 22 2015, 5:21 PM
araujo mentioned this in rS281879: Missing break in switch case..Apr 23 2015, 2:50 AM
araujo closed this revision.Apr 23 2015, 2:50 AM

Committed.. Thanks!
I usually keep checking the CID report, in case I find something else, I will let you know.

All the best.

Revision Contents
Changeset List

PathSize
sys/
amd64/
vmm/
amd/
1 line

Diff 4946

View Options

sys/amd64/vmm/amd/svm.c

Loading...