On 2/27/2016 9:54 PM, Colin King wrote:
From: Colin Ian King <[email protected]>

The boolean 'found' is not initialized and hence garbage. It should
be initialized as false.

Found with static analysis using CoverityScan

Signed-off-by: Colin Ian King <[email protected]>

Acked-by: Milo Kim <[email protected]>

Thanks for catching this.

---
  drivers/power/lp8788-charger.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/lp8788-charger.c b/drivers/power/lp8788-charger.c
index f5a48fd..7321b72 100644
--- a/drivers/power/lp8788-charger.c
+++ b/drivers/power/lp8788-charger.c
@@ -455,7 +455,7 @@ static void lp8788_charger_event(struct work_struct *work)

  static bool lp8788_find_irq_id(struct lp8788_charger *pchg, int virq, int *id)
  {
-       bool found;
+       bool found = false;
        int i;

        for (i = 0; i < pchg->num_irqs; i++) {

Reply via email to