logger: adapt access mode checks

Allow setting the tag and the priority only for files opened O_WRONLY.

Change-Id: Icc9d7a8eb96f8f1b1aa00717aae1f32f7a2e52e4
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

diff --git a/drivers/staging/android/logger.c b/drivers/staging/android/logger.c
index fb6afb3..67608e0 100644 (file)
--- a/drivers/staging/android/logger.c
+++ b/drivers/staging/android/logger.c
@@ -957,7 +957,8 @@ static long logger_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                ret = logger_set_version(reader, argp);
                break;
        case LOGGER_SET_PRIO:   /* 44552 */
-               if (file->f_mode & FMODE_READ) {
+               if ((file->f_mode & FMODE_READ) ||
+                   !(file->f_mode & FMODE_WRITE)) {
                        ret = -EBADF;
                        break;
                }
@@ -965,7 +966,8 @@ static long logger_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                ret = logger_set_prio(writer, argp);
                break;
        case LOGGER_SET_TAG:    /* 44551 */
-               if (file->f_mode & FMODE_READ) {
+               if ((file->f_mode & FMODE_READ) ||
+                   !(file->f_mode & FMODE_WRITE)) {
                        ret = -EBADF;
                        break;
                }