Re: [PATCH] modutils: ieee1394 device_id extraction

Artur Frysiak (wiget@pld.org.pl)
Mon, 17 Sep 2001 13:15:40 +0200


On Sun, Sep 16, 2001 at 12:18:53PM +0200, Kristian Hogsberg wrote:
> @@ -1225,6 +1306,8 @@
> if (usbmap != stdout)
> fclose(usbmap);
> if (parportmap != stdout)
> + fclose(parportmap);
> + if (ieee1394map != stdout)
> fclose(parportmap);
> /* Close depfile last, it's timestamp is critical */
> if (dep != stdout)

I think, this part is bad. You test for ieee139map file pointer and close
parportmap.

Regards

-- 
Artur Frysiak
http://www.pld.org.pl/
-
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/