diff options
author | Xiaochen Wang <wangxiaochen0@gmail.com> | 2011-03-18 04:29:25 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2011-03-18 10:14:57 -0400 |
commit | 977a6ef3c0cb622b572fb8e2e5088dbe09521375 (patch) | |
tree | 561b8544ad980f3ab3fd19f7475a304aea1f94bf /sound/sound_firmware.c | |
parent | d351cf4603edb2a5bfa9a48d06c425511c63f2a3 (diff) |
sound: read i_size with i_size_read()
Convert direct read of inode->i_size to using i_size_read().
i_size_read is guaranteed to return a valid value and
its caller does not need to use addtional locking.
Signed-off-by: Xiaochen Wang <wangxiaochen0@gmail.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/sound_firmware.c')
-rw-r--r-- | sound/sound_firmware.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/sound_firmware.c b/sound/sound_firmware.c index 340a0bc5303e..7e96249536b4 100644 --- a/sound/sound_firmware.c +++ b/sound/sound_firmware.c | |||
@@ -19,7 +19,7 @@ static int do_mod_firmware_load(const char *fn, char **fp) | |||
19 | printk(KERN_INFO "Unable to load '%s'.\n", fn); | 19 | printk(KERN_INFO "Unable to load '%s'.\n", fn); |
20 | return 0; | 20 | return 0; |
21 | } | 21 | } |
22 | l = filp->f_path.dentry->d_inode->i_size; | 22 | l = i_size_read(filp->f_path.dentry->d_inode); |
23 | if (l <= 0 || l > 131072) | 23 | if (l <= 0 || l > 131072) |
24 | { | 24 | { |
25 | printk(KERN_INFO "Invalid firmware '%s'\n", fn); | 25 | printk(KERN_INFO "Invalid firmware '%s'\n", fn); |