linux-kernel - [PATCH] drivers/block: Remove unnecessary cast

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:	Fri, 8 Jan 2010 11:22:33 -0700
From:	H Hartley Sweeten <hartleys@...ionengravers.com>
To:	Linux Kernel <linux-kernel@...r.kernel.org>, iss_storagedev@...com
Cc:	chirag.kantharia@...com
Subject: [PATCH] drivers/block: Remove unnecessary cast

drivers/block: Remove unnecessary cast.

The struct seq_file 'private' member is a void *, the cast is not needed.

Signed-off-by: H Hartley Sweeten <hsweeten@...ionengravers.com>
Cc: Chirag Kantharia <chirag.kantharia@...com>

---

diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
index ce1fa92..bd85fb8 100644
--- a/drivers/block/DAC960.c
+++ b/drivers/block/DAC960.c
@@ -6458,7 +6458,7 @@ static const struct file_operations dac960_proc_fops = {
 
 static int dac960_initial_status_proc_show(struct seq_file *m, void *v)
 {
-	DAC960_Controller_T *Controller = (DAC960_Controller_T *)m->private;
+	DAC960_Controller_T *Controller = m->private;
 	seq_printf(m, "%.*s", Controller->InitialStatusLength, Controller->CombinedStatusBuffer);
 	return 0;
 }
@@ -6478,7 +6478,7 @@ static const struct file_operations dac960_initial_status_proc_fops = {
 
 static int dac960_current_status_proc_show(struct seq_file *m, void *v)
 {
-  DAC960_Controller_T *Controller = (DAC960_Controller_T *) m->private;
+  DAC960_Controller_T *Controller = m->private;
   unsigned char *StatusMessage =
     "No Rebuild or Consistency Check in Progress\n";
   int ProgressMessageLength = strlen(StatusMessage);
@@ -6524,7 +6524,7 @@ static const struct file_operations dac960_current_status_proc_fops = {
 
 static int dac960_user_command_proc_show(struct seq_file *m, void *v)
 {
-	DAC960_Controller_T *Controller = (DAC960_Controller_T *)m->private;
+	DAC960_Controller_T *Controller = m->private;
 
 	seq_printf(m, "%.*s", Controller->UserStatusLength, Controller->UserStatusBuffer);
 	return 0;
diff --git a/drivers/block/cpqarray.c b/drivers/block/cpqarray.c
index 6422651..9bcef1a 100644
--- a/drivers/block/cpqarray.c
+++ b/drivers/block/cpqarray.c
@@ -228,7 +228,7 @@ static void __init ida_procinit(int i)
 static int ida_proc_show(struct seq_file *m, void *v)
 {
 	int i, ctlr;
-	ctlr_info_t *h = (ctlr_info_t*)m->private;
+	ctlr_info_t *h = m->private;
 	drv_info_t *drv;
 #ifdef CPQ_PROC_PRINT_QUEUES
 	cmdlist_t *c;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/