[PATCH] intel, gma500, lvds: Fix use after free on psb_intel_lvds_init()

2012-01-14 Thread Jesper Juhl
On Sat, 14 Jan 2012, Jesper Juhl wrote: > In psb_intel_lvds_init(), if we fail to allocate memory for > 'psb_intel_connector' we free the memory we previously allocated for > 'psb_intel_encoder', but we then proceed to use that free'd pointer > when we do 'psb_intel_encoder->dev_priv =

[PATCH] intel, gma500, lvds: Fix use after free on psb_intel_lvds_init()

2012-01-14 Thread Jesper Juhl
In psb_intel_lvds_init(), if we fail to allocate memory for 'psb_intel_connector' we free the memory we previously allocated for 'psb_intel_encoder', but we then proceed to use that free'd pointer when we do 'psb_intel_encoder->dev_priv = lvds_priv;'. I believe the proper way to handle this is to

[PATCH] intel, gma500, lvds: Fix use after free on psb_intel_lvds_init()

2012-01-14 Thread Jesper Juhl
In psb_intel_lvds_init(), if we fail to allocate memory for 'psb_intel_connector' we free the memory we previously allocated for 'psb_intel_encoder', but we then proceed to use that free'd pointer when we do 'psb_intel_encoder-dev_priv = lvds_priv;'. I believe the proper way to handle this is to

Re: [PATCH] intel, gma500, lvds: Fix use after free on psb_intel_lvds_init()

2012-01-14 Thread Jesper Juhl
On Sat, 14 Jan 2012, Jesper Juhl wrote: In psb_intel_lvds_init(), if we fail to allocate memory for 'psb_intel_connector' we free the memory we previously allocated for 'psb_intel_encoder', but we then proceed to use that free'd pointer when we do 'psb_intel_encoder-dev_priv = lvds_priv;'.