[Xcb] [Xcb-commit] icccm

[Alan Coopersmith]

Vincent Torri wrote:
> On Thu, 24 Jun 2010, Alan Coopersmith wrote:
>> +  if(!reply || reply->type == XCB_NONE) {
>> +    free(reply);
>>     return 0;
>> +  }
> 
> there will be a crash if reply is NULL. check reply before freeing it.

Not on any implementation of free() that's compliant with ANSI/ISO C89,
which requires that free(NULL) simply return safely.

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System