Page MenuHomeFreeBSD
Differential D5250

netfront: remove pointless assignment in xn_ioctl
ClosedPublic
Actions

Authored by liuw_liuw.name on Feb 11 2016, 11:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 17, 6:39 PM
Unknown Object (File)
Fri, Jan 17, 1:37 PM
Unknown Object (File)
Wed, Jan 15, 3:24 PM
Unknown Object (File)
Wed, Jan 8, 12:07 AM
Unknown Object (File)
Dec 22 2024, 7:00 AM
Unknown Object (File)
Dec 1 2024, 9:23 PM
Unknown Object (File)
Dec 1 2024, 9:23 PM
Unknown Object (File)
Dec 1 2024, 9:22 PM
View All 76 Files
Subscribers
imp

Details

Reviewers
royger
Commits
rS295510: xen-netfront: remove pointless assignment in xn_ioctl
Summary

The variable error is assigned to 0 before entering the switch.
Assigning error to 0 before break pointless rewrites the real error
value that should be returned.

This is CID 1304974.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2459
Build 2476: arc lint + arc unit

Event Timeline

liuw_liuw.name updated this revision to Diff 13212.Feb 11 2016, 11:39 AM
liuw_liuw.name retitled this revision from to netfront: remove pointless assignment in xn_ioctl.
liuw_liuw.name updated this object.
liuw_liuw.name edited the test plan for this revision. (Show Details)
liuw_liuw.name added a reviewer: royger.
Herald added a subscriber: imp. · View Herald TranscriptFeb 11 2016, 11:39 AM
royger accepted this revision.Feb 11 2016, 11:46 AM
royger edited edge metadata.

LGTM

This revision is now accepted and ready to land.Feb 11 2016, 11:46 AM
Closed by commit rS295510: xen-netfront: remove pointless assignment in xn_ioctl (authored by royger). · Explain WhyFeb 11 2016, 11:50 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
dev/
xen/
netfront/
2 lines

Diff 13212

View Options

sys/dev/xen/netfront/netfront.c

Loading...