[PATCH 4/4] video/imx: always initialize offscreenbuf member

Daniel Mierswa impulze at impulze.org
Thu Jan 17 08:04:22 EST 2013


On 01/17/2013 12:56 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 07:32 Thu 17 Jan     , Daniel Mierswa wrote:
>> +	/*
>> +	 * sc.offscreenbuf is dangling here, if no offscreen buffer was
>> +	 * requested, so set it to NULL otherwise the free in fd_close
>> +	 * will cause undefined behaviour
>> +	 */
>> +	if (!offscreen)
>> +		sc.offscreenbuf = NULL;
>> +
> NAck
> 
> this is already done by memset
> 
> Best Regards,
> J.
>>  	if (sc.offscreenbuf) {
>>  		if (do_bg)
>>  			memset_pixel(&info, sc.offscreenbuf, bg_color,

Huh? I don't get it.

-- 
Mierswa, Daniel

If you still don't like it, that's ok: that's why I'm boss. I simply know better than you do.
               --- Linus Torvalds, comp.os.linux.advocacy, 1996/07/22



More information about the barebox mailing list