diff options
| author | Maarten Lankhorst <maarten.lankhorst@canonical.com> | 2013-03-24 15:36:38 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-12 09:52:04 -0700 |
| commit | 2705de01e2dcd27e86aafc8837998c885f9cfcef (patch) | |
| tree | b99958bbb20e39fb16b71df00e3489bb4e44012b /security/apparmor/lib.c | |
| parent | 05c05ef21a1cf71d16189cd4d8060b47f504db2b (diff) | |
| download | lwn-2705de01e2dcd27e86aafc8837998c885f9cfcef.tar.gz lwn-2705de01e2dcd27e86aafc8837998c885f9cfcef.zip | |
drm/nouveau: fix handling empty channel list in ioctl's
commit b43decd2c513ae44a5462a1a4c2622f0fa77c9e0 upstream.
If there are no channels, chan would never end up being NULL,
and so the null pointer check would fail.
Solve this by initializing chan to NULL, and iterating over temp instead.
Fixes oops when running intel-gpu-tools/tests/kms_flip, which attempts to
do some intel ioctl's on a nouveau device.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'security/apparmor/lib.c')
0 files changed, 0 insertions, 0 deletions