Commit c3b65713 authored by Jeff Garzik's avatar Jeff Garzik Committed by Linus Torvalds

[PATCH] fs/eventpoll: error handling micro-cleanup

While reviewing the 'may be used uninitialized' bogus gcc warnings, I
noticed that an error code assignment was only needed if an error had
actually occured.
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
Cc: Davide Libenzi <davidel@xmailserver.org>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent e1ca66d1
...@@ -720,9 +720,10 @@ static int ep_getfd(int *efd, struct inode **einode, struct file **efile, ...@@ -720,9 +720,10 @@ static int ep_getfd(int *efd, struct inode **einode, struct file **efile,
/* Allocates an inode from the eventpoll file system */ /* Allocates an inode from the eventpoll file system */
inode = ep_eventpoll_inode(); inode = ep_eventpoll_inode();
error = PTR_ERR(inode); if (IS_ERR(inode)) {
if (IS_ERR(inode)) error = PTR_ERR(inode);
goto eexit_2; goto eexit_2;
}
/* Allocates a free descriptor to plug the file onto */ /* Allocates a free descriptor to plug the file onto */
error = get_unused_fd(); error = get_unused_fd();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment