Hi Harald,

Is the memory you allocate freed anywhere?  We have a large
number of memory leaks in the front end, but maybe we shouldn't
add another one :-)

my reading of the definition of XALLOCAVEC

include/libiberty.h:#define XALLOCAVEC(T, N)    ((T *) alloca (sizeof (T) * 
(N)))

and alloca(3):

ALLOCA(3)                       Linux Programmer's Manual                      
ALLOCA(3)

You're right, I misread that.

So, the patch is OK, and thanks for the explanation!

Best regards

        Thomas

Reply via email to