diff options
author | Jeff Garzik <jeff@garzik.org> | 2006-10-03 04:13:52 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-03 11:03:41 -0400 |
commit | c3b6571384024be31e8b3442c0e7c3b127110d9b (patch) | |
tree | 24281e244579d97eb944d7b48e0036f27db4fe3c /fs | |
parent | e1ca66d1b990b23e7753c729332c0ada61f4f38d (diff) |
[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: Jeff Garzik <jeff@garzik.org>
Cc: Davide Libenzi <davidel@xmailserver.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/eventpoll.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/eventpoll.c b/fs/eventpoll.c index 8d544334bcd2..557d5b614fae 100644 --- a/fs/eventpoll.c +++ b/fs/eventpoll.c | |||
@@ -720,9 +720,10 @@ static int ep_getfd(int *efd, struct inode **einode, struct file **efile, | |||
720 | 720 | ||
721 | /* Allocates an inode from the eventpoll file system */ | 721 | /* Allocates an inode from the eventpoll file system */ |
722 | inode = ep_eventpoll_inode(); | 722 | inode = ep_eventpoll_inode(); |
723 | error = PTR_ERR(inode); | 723 | if (IS_ERR(inode)) { |
724 | if (IS_ERR(inode)) | 724 | error = PTR_ERR(inode); |
725 | goto eexit_2; | 725 | goto eexit_2; |
726 | } | ||
726 | 727 | ||
727 | /* Allocates a free descriptor to plug the file onto */ | 728 | /* Allocates a free descriptor to plug the file onto */ |
728 | error = get_unused_fd(); | 729 | error = get_unused_fd(); |