On Sun, 01 Jun 2014 16:20:44 +0200
Christoph Lohmann <[email protected]> wrote:

> The  xsetcolorname() function should be conservative in its input. Since
> it is only called from escape input it shouldn’t overwrite a  dc.col  if
> something  failed. I fixed this and refactored to the function according
> to your patch, which removed the temporary variables.

It wouldn't overwrite dc.col's adress in case something fails
internally, but it is a memleak the way I did it, because dc.col is not
freed before being re-assigned.

Cheers

FRIGN

-- 
FRIGN <[email protected]>

Reply via email to