Page MenuHomeFreeBSD
Differential D45272

sound: Prevent uninitialized variable destruction in chn_init()
ClosedPublic
Actions

Authored by christos on Mon, May 20, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 22, 7:35 PM
Unknown Object (File)
Wed, May 22, 10:04 AM
Unknown Object (File)
Wed, May 22, 3:58 AM
Unknown Object (File)
Wed, May 22, 3:26 AM
Unknown Object (File)
Tue, May 21, 8:55 PM
Unknown Object (File)
Tue, May 21, 8:53 AM
Unknown Object (File)
Tue, May 21, 8:50 AM
Unknown Object (File)
Mon, May 20, 2:27 PM
Subscribers
imp

Details

Reviewers
markj
emaste
Commits
rGbca6f391aed9: sound: Prevent uninitialized variable destruction in chn_init()
rG5d1a5d6f1f59: sound: Prevent uninitialized variable destruction in chn_init()
Summary

If dsp_unit2name() fails, we'll get to out2 with b, bs and devinfo
uninitialized, which will result in a panic.

Reported by: Pierre Pronchery <pierre@freebsdfoundation.org>
Reported by: Coverity Scan
CID: 1545029, 1545025
Pull-request: https://github.com/freebsd/freebsd-src/pull/1240
Sponsored by: The FreeBSD Foundation
MFC after: 1 day

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

christos created this revision.Mon, May 20, 2:03 PM
Herald added a subscriber: imp. · View Herald TranscriptMon, May 20, 2:03 PM
christos requested review of this revision.Mon, May 20, 2:03 PM
Harbormaster completed remote builds in B57790: Diff 138784.Mon, May 20, 2:03 PM
markj accepted this revision.Mon, May 20, 2:08 PM
markj added inline comments.
sys/dev/sound/pcm/channel.c
1252

These assignments are redundant since c was allocated with M_ZERO. You could just get rid of them.

This revision is now accepted and ready to land.Mon, May 20, 2:08 PM
christos marked an inline comment as done.Mon, May 20, 2:12 PM
christos added inline comments.
sys/dev/sound/pcm/channel.c
1252

D45273

christos updated this revision to Diff 138786.Mon, May 20, 2:12 PM
christos marked an inline comment as done.

Address Mark's comment.

This revision now requires review to proceed.Mon, May 20, 2:12 PM
Harbormaster completed remote builds in B57792: Diff 138786.Mon, May 20, 2:12 PM
christos added a parent revision: D45273: sound: Get rid of redundant assignments in chn_init().Mon, May 20, 2:12 PM
markj added inline comments.Mon, May 20, 2:13 PM
sys/dev/sound/pcm/channel.c
1252

It should just be done in a single patch, IMHO, but ok.

emaste accepted this revision.Mon, May 20, 2:15 PM
This revision is now accepted and ready to land.Mon, May 20, 2:15 PM
Closed by commit rG5d1a5d6f1f59: sound: Prevent uninitialized variable destruction in chn_init() (authored by christos). · Explain WhyMon, May 20, 2:15 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG5d1a5d6f1f59: sound: Prevent uninitialized variable destruction in chn_init().
christos added a commit: rGbca6f391aed9: sound: Prevent uninitialized variable destruction in chn_init().Tue, May 21, 5:54 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
4 lines

Diff 138788

View Options

sys/dev/sound/pcm/channel.c

Loading...