diff options
author | Josh Boyer <jwboyer@fedoraproject.org> | 2014-04-25 08:31:16 -0400 |
---|---|---|
committer | Josh Boyer <jwboyer@fedoraproject.org> | 2014-04-25 08:31:24 -0400 |
commit | 107655b9b2f2bc8604bfc2066df02b8266ea35f5 (patch) | |
tree | b38ec9714d33847032f87e6ab97ab63ad50ceaf4 | |
parent | 508411e608d4c03dd0c40c97aba0e46b369b0983 (diff) | |
download | kernel-107655b9b2f2bc8604bfc2066df02b8266ea35f5.tar.gz kernel-107655b9b2f2bc8604bfc2066df02b8266ea35f5.tar.xz kernel-107655b9b2f2bc8604bfc2066df02b8266ea35f5.zip |
Add patch from Will Woods to fix fanotify EOVERFLOW issue (rhbz 696821)
-rw-r--r-- | fanotify-fix-EOVERFLOW-on-64-bit.patch | 32 | ||||
-rw-r--r-- | kernel.spec | 7 |
2 files changed, 39 insertions, 0 deletions
diff --git a/fanotify-fix-EOVERFLOW-on-64-bit.patch b/fanotify-fix-EOVERFLOW-on-64-bit.patch new file mode 100644 index 00000000..11ba4e1c --- /dev/null +++ b/fanotify-fix-EOVERFLOW-on-64-bit.patch @@ -0,0 +1,32 @@ +Bugzilla: 696821 +Upstream-status: Sent http://marc.info/?l=linux-kernel&m=139835974112096&w=2 + +On 64-bit systems, O_LARGEFILE is automatically added to flags inside +the open() syscall (also openat(), blkdev_open(), etc). +Userspace therefore defines O_LARGEFILE to be 0 - you can use it, +but it's a no-op. Everything should be O_LARGEFILE by default. + +But: when fanotify does create_fd() it uses dentry_open(), which skips +all that. And userspace can't set O_LARGEFILE in fanotify_init() because +it's defined to 0. So if fanotify gets an event regarding a large file, +the read() will just fail with -EOVERFLOW. + +This patch adds O_LARGEFILE to fanotify_init()'s event_f_flags on 64-bit +systems, using the same test as open()/openat()/etc. + +Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=696821 + +Acked-by: Eric Paris <eparis@redhat.com> +Signed-off-by: Will Woods <wwoods@redhat.com> + +--- linux-3.14.1-200.fc20.x86_64/fs/notify/fanotify/fanotify_user.c.orig 2014-04-23 18:15:29.347644932 -0400 ++++ linux-3.14.1-200.fc20.x86_64/fs/notify/fanotify/fanotify_user.c 2014-04-23 18:17:44.249438484 -0400 +@@ -742,6 +742,8 @@ + oevent->path.mnt = NULL; + oevent->path.dentry = NULL; + ++ if (force_o_largefile()) ++ event_f_flags |= O_LARGEFILE; + group->fanotify_data.f_flags = event_f_flags; + #ifdef CONFIG_FANOTIFY_ACCESS_PERMISSIONS + oevent->response = 0; diff --git a/kernel.spec b/kernel.spec index 147ef010..59e46c05 100644 --- a/kernel.spec +++ b/kernel.spec @@ -745,6 +745,9 @@ Patch25066: 0001-synaptics-Add-min-max-quirk-for-ThinkPad-Edge-E431.patch #rhbz 1090746 Patch25067: ACPICA-Tables-Fix-bad-pointer-issue-in-acpi_tb_parse_root_table.patch +#rhbz 696821 +Patch25068: fanotify-fix-EOVERFLOW-on-64-bit.patch + # END OF PATCH DEFINITIONS %endif @@ -1444,6 +1447,9 @@ ApplyPatch 0001-synaptics-Add-min-max-quirk-for-ThinkPad-Edge-E431.patch #rhbz 1090746 ApplyPatch ACPICA-Tables-Fix-bad-pointer-issue-in-acpi_tb_parse_root_table.patch +#rhbz 696821 +ApplyPatch fanotify-fix-EOVERFLOW-on-64-bit.patch + # END OF PATCH APPLICATIONS %endif @@ -2256,6 +2262,7 @@ fi # || || %changelog * Fri Apr 25 2014 Josh Boyer <jwboyer@fedoraproject.org> +- Add patch from Will Woods to fix fanotify EOVERFLOW issue (rhbz 696821) - Fix ACPI issue preventing boot on AMI firmware (rhbz 1090746) * Fri Apr 25 2014 Hans de Goede <hdegoede@redhat.com> |