GitHub-Mirror/Star64_linux
2
0
Fork 0
mirror of https://github.com/Fishwaldo/Star64_linux.git synced 2025-04-21 13:53:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

pcm_native: switch to fdget()/fdput()

Browse source 
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
Al Viro 2013-06-05 14:09:55 -04:00
parent ac6614b764
commit 0888c321de
1 changed files with 15 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

40
sound/core/pcm_native.c
View file

@ -1589,29 +1589,16 @@ static int snd_pcm_drop(struct snd_pcm_substream *substream)
} }
/* WARNING: Don't forget to fput back the file */ static bool is_pcm_file(struct file *file)
static struct file *snd_pcm_file_fd(int fd, int *fput_needed)
{ {
struct file *file; struct inode *inode = file_inode(file);
struct inode *inode;
unsigned int minor; unsigned int minor;
file = fget_light(fd, fput_needed); if (!S_ISCHR(inode->i_mode) || imajor(inode) != snd_major)
if (!file) return false;
return NULL;
inode = file_inode(file);
if (!S_ISCHR(inode->i_mode) ||
imajor(inode) != snd_major) {
fput_light(file, *fput_needed);
return NULL;
}
minor = iminor(inode); minor = iminor(inode);
if (!snd_lookup_minor_data(minor, SNDRV_DEVICE_TYPE_PCM_PLAYBACK) && return snd_lookup_minor_data(minor, SNDRV_DEVICE_TYPE_PCM_PLAYBACK) ||
!snd_lookup_minor_data(minor, SNDRV_DEVICE_TYPE_PCM_CAPTURE)) { snd_lookup_minor_data(minor, SNDRV_DEVICE_TYPE_PCM_CAPTURE);
fput_light(file, *fput_needed);
return NULL;
}
return file;
} }
/* /*
@ -1620,16 +1607,18 @@ static struct file *snd_pcm_file_fd(int fd, int *fput_needed)
static int snd_pcm_link(struct snd_pcm_substream *substream, int fd) static int snd_pcm_link(struct snd_pcm_substream *substream, int fd)
{ {
int res = 0; int res = 0;
struct file *file;
struct snd_pcm_file *pcm_file; struct snd_pcm_file *pcm_file;
struct snd_pcm_substream *substream1; struct snd_pcm_substream *substream1;
struct snd_pcm_group *group; struct snd_pcm_group *group;
int fput_needed; struct fd f = fdget(fd);
file = snd_pcm_file_fd(fd, &fput_needed); if (!f.file)
if (!file)
return -EBADFD; return -EBADFD;
pcm_file = file->private_data; if (!is_pcm_file(f.file)) {
res = -EBADFD;
goto _badf;
}
pcm_file = f.file->private_data;
substream1 = pcm_file->substream; substream1 = pcm_file->substream;
group = kmalloc(sizeof(*group), GFP_KERNEL); group = kmalloc(sizeof(*group), GFP_KERNEL);
if (!group) { if (!group) {
@ -1663,8 +1652,9 @@ static int snd_pcm_link(struct snd_pcm_substream *substream, int fd)
up_write(&snd_pcm_link_rwsem); up_write(&snd_pcm_link_rwsem);
_nolock: _nolock:
snd_card_unref(substream1->pcm->card); snd_card_unref(substream1->pcm->card);
fput_light(file, fput_needed);
kfree(group); kfree(group);
_badf:
fdput(f);
return res; return res;
} }