sysfs: Complain bitterly about attempts to remove files from nonexistent directories.
commitbfd534c72bec682cbf1017e54f0749aac24840eb
authorEric W. Biederman <ebiederm@xmission.com>
Sat, 14 Jan 2012 05:32:59 +0000 (13 21:32 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 3 Feb 2012 17:21:47 +0000 (3 09:21 -0800)
tree567c1ebded97b8662c83b3a365184401e3795932
parent33d812de490956625ef74e2fb47d6dc2c9d34c0f
sysfs: Complain bitterly about attempts to remove files from nonexistent directories.

commit ce597919361dcec97341151690e780eade2a9cf4 upstream.

Recently an OOPS was observed from the usb serial io_ti driver when it tried to remove
sysfs directories.  Upon investigation it turns out this driver was always buggy
and that a recent sysfs change had stopped guarding itself against removing attributes
from sysfs directories that had already been removed. :(

Historically we have been silent about attempting to files from nonexistent sysfs
directories and have politely returned error codes.  That has resulted in people writing
broken code that ignores the error codes.

Issue a kernel WARNING and a stack backtrace to make it clear in no uncertain
terms that abusing sysfs is not ok, and the callers need to fix their code.

This change transforms the io_ti OOPS into a more comprehensible error message
and stack backtrace.

Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Reported-by: Wolfgang Frisch <wfpub@roembden.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/sysfs/file.c
fs/sysfs/inode.c