Staging: comedi: initialize divisor variables

The i8253_cascade_ns_to_timer_2div() function (and
i8253_cascade_ns_to_timer macro) checks the old values *d1 and *d2
for correctness as a heuristic before calculating new values.  Don't
call the function with uninitialized values in *d1 and *d2.

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

diff --git a/drivers/staging/comedi/drivers/adl_pci9118.c b/drivers/staging/comedi/drivers/adl_pci9118.c
index 1ee4b6a91c1fbab1807603f2d9bd248327b79cdd..791ea8334e1e0f2ab50f9c1055f3fdc179e8f4eb 100644 (file)
--- a/drivers/staging/comedi/drivers/adl_pci9118.c
+++ b/drivers/staging/comedi/drivers/adl_pci9118.c
@@ -780,7 +780,7 @@ static int pci9118_ai_cmdtest(struct comedi_device *dev,
                              struct comedi_cmd *cmd)
 {
        int err = 0;
-       int tmp, divisor1, divisor2;
+       int tmp, divisor1 = 0, divisor2 = 0;
 
        /* step 1: make sure trigger sources are trivially valid */
 

diff --git a/drivers/staging/comedi/drivers/adv_pci1710.c b/drivers/staging/comedi/drivers/adv_pci1710.c
index f0ae4c06fe9554c1803bac6e90217d94e0633d47..951e57949f7fdfde4cba3a50aa21b8d428343b62 100644 (file)
--- a/drivers/staging/comedi/drivers/adv_pci1710.c
+++ b/drivers/staging/comedi/drivers/adv_pci1710.c
@@ -830,7 +830,7 @@ static irqreturn_t interrupt_service_pci1710(int irq, void *d)
 static int pci171x_ai_docmd_and_mode(int mode, struct comedi_device *dev,
                                     struct comedi_subdevice *s)
 {
-       unsigned int divisor1, divisor2;
+       unsigned int divisor1 = 0, divisor2 = 0;
        unsigned int seglen;
 
        DPRINTK("adv_pci1710 EDBG: BGN: pci171x_ai_docmd_and_mode(%d,...)\n",
@@ -934,7 +934,7 @@ static int pci171x_ai_cmdtest(struct comedi_device *dev,
                              struct comedi_cmd *cmd)
 {
        int err = 0;
-       int tmp, divisor1, divisor2;
+       int tmp, divisor1 = 0, divisor2 = 0;
 
        DPRINTK("adv_pci1710 EDBG: BGN: pci171x_ai_cmdtest(...)\n");
 #ifdef PCI171X_EXTDEBUG

diff --git a/drivers/staging/comedi/drivers/cb_das16_cs.c b/drivers/staging/comedi/drivers/cb_das16_cs.c
index b47b3b1ba3501dea171409bd7fd5627f54060cb9..bc375e73abc105cdc6a3523d2c7e1c7583f13046 100644 (file)
--- a/drivers/staging/comedi/drivers/cb_das16_cs.c
+++ b/drivers/staging/comedi/drivers/cb_das16_cs.c
@@ -452,7 +452,7 @@ static int das16cs_ai_cmdtest(struct comedi_device *dev,
        /* step 4: fix up any arguments */
 
        if (cmd->scan_begin_src == TRIG_TIMER) {
-               unsigned int div1, div2;
+               unsigned int div1 = 0, div2 = 0;
 
                tmp = cmd->scan_begin_arg;
                i8253_cascade_ns_to_timer(100, &div1, &div2,
@@ -462,7 +462,7 @@ static int das16cs_ai_cmdtest(struct comedi_device *dev,
                        err++;
        }
        if (cmd->convert_src == TRIG_TIMER) {
-               unsigned int div1, div2;
+               unsigned int div1 = 0, div2 = 0;
 
                tmp = cmd->convert_arg;
                i8253_cascade_ns_to_timer(100, &div1, &div2,

diff --git a/drivers/staging/comedi/drivers/pcl711.c b/drivers/staging/comedi/drivers/pcl711.c
index dd9db069a932469f85aae339b842b8faef4fdb1f..4914784f6995d121a76e028474ce6eb2ec9076e2 100644 (file)
--- a/drivers/staging/comedi/drivers/pcl711.c
+++ b/drivers/staging/comedi/drivers/pcl711.c
@@ -407,6 +407,7 @@ static int pcl711_ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
                 *  0xb4 = Select Counter 2 | LSB/MSB | Mode=2 | Binary
                 */
 
+               timer1 = timer2 = 0;
                i8253_cascade_ns_to_timer(i8253_osc_base, &timer1, &timer2,
                                          &cmd->scan_begin_arg,
                                          TRIG_ROUND_NEAREST);

diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comedi/drivers/pcl816.c
index fa2414500a07fe1b16879cf2ab98b78f4e5f4294..f9a0dac79eca40531490416ec89566c41d6257e6 100644 (file)
--- a/drivers/staging/comedi/drivers/pcl816.c
+++ b/drivers/staging/comedi/drivers/pcl816.c
@@ -472,7 +472,7 @@ static int pcl816_ai_cmdtest(struct comedi_device *dev,
                             struct comedi_subdevice *s, struct comedi_cmd *cmd)
 {
        int err = 0;
-       int tmp, divisor1, divisor2;
+       int tmp, divisor1 = 0, divisor2 = 0;
 
        DEBUG(printk("pcl816 pcl812_ai_cmdtest\n"); pcl816_cmdtest_out(-1, cmd);
            );

diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
index e95229b13118e154ae6b63a4657fec3c4c3ebd67..819dd838b075a74dafa7d729a66bec81610ca911 100644 (file)
--- a/drivers/staging/comedi/drivers/pcl818.c
+++ b/drivers/staging/comedi/drivers/pcl818.c
@@ -970,7 +970,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
                              struct comedi_subdevice *s)
 {
        struct comedi_cmd *cmd = &s->async->cmd;
-       int divisor1, divisor2;
+       int divisor1 = 0, divisor2 = 0;
        unsigned int seglen;
 
        printk("pcl818_ai_cmd_mode()\n");
@@ -1089,7 +1089,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 static int pcl818_ao_mode13(int mode, struct comedi_device *dev,
                            struct comedi_subdevice *s, comedi_trig * it)
 {
-       int divisor1, divisor2;
+       int divisor1 = 0, divisor2 = 0;
 
        if (!dev->irq) {
                comedi_error(dev, "IRQ not defined!");
@@ -1287,7 +1287,7 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
                      struct comedi_cmd *cmd)
 {
        int err = 0;
-       int tmp, divisor1, divisor2;
+       int tmp, divisor1 = 0, divisor2 = 0;
 
        /* step 1: make sure trigger sources are trivially valid */