Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1173835

[RFC PATCH v2 3/4] ppdev: add compat ioctl

From Bamvor Zhang Jian <bamvor.zhangjian@linaro.org>
Newsgroups linux.kernel
Subject [RFC PATCH v2 3/4] ppdev: add compat ioctl
Date 2015-06-29 16:30 +0200
Message-ID <pGIem-5zg-17@gated-at.bofh.it> (permalink)
References <pGIel-5zg-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Add compat ioctl in ppdev in order to solve the y2038 issue in
later patch.
This patch simply add pp_do_ioctl to compat_ioctl, because I found
that all the ioctl access the arg as a pointer.

Signed-off-by: Bamvor Zhang Jian <bamvor.zhangjian@linaro.org>
---
 drivers/char/ppdev.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/char/ppdev.c b/drivers/char/ppdev.c
index ae0b42b..9207658 100644
--- a/drivers/char/ppdev.c
+++ b/drivers/char/ppdev.c
@@ -69,6 +69,7 @@
 #include <linux/ppdev.h>
 #include <linux/mutex.h>
 #include <linux/uaccess.h>
+#include <linux/compat.h>
 
 #define PP_VERSION "ppdev: user-space parallel port driver"
 #define CHRDEV "ppdev"
@@ -635,6 +636,11 @@ static long pp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
 	return ret;
 }
 
+static long pp_compat_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
+{
+	return pp_ioctl(file, cmd, (unsigned long)compat_ptr(arg));
+}
+
 static int pp_open (struct inode * inode, struct file * file)
 {
 	unsigned int minor = iminor(inode);
@@ -744,6 +750,9 @@ static const struct file_operations pp_fops = {
 	.write		= pp_write,
 	.poll		= pp_poll,
 	.unlocked_ioctl	= pp_ioctl,
+#ifdef CONFIG_COMPAT
+	.compat_ioctl	= pp_compat_ioctl,
+#endif
 	.open		= pp_open,
 	.release	= pp_release,
 };
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[RFC PATCH v2 0/4] Convert ppdev to y2038 safe Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> - 2015-06-29 16:30 +0200
  [RFC PATCH v2 2/4] time64: add timeval64 helper for compat syscalls Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> - 2015-06-29 16:30 +0200
  [RFC PATCH v2 3/4] ppdev: add compat ioctl Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> - 2015-06-29 16:30 +0200
  [RFC PATCH v2 4/4] y2038: convert ppdev to 2038 safe Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> - 2015-06-29 16:30 +0200

csiph-web