Danny,

You are correct. I looked at the stack trace and it is from 2.2.7. The 
problem was fixed after that as the call is also protected in 2.3.4.







From:   Danny Auble <d...@schedmd.com>
To:     "slurm-dev" <slurm-dev@schedmd.com>, 
Date:   03/26/2012 11:53 AM
Subject:        [slurm-dev] Re: Segfault in Task Affinity



Rod,

While this is probably a better way to handle what is going on, the only 
call to _lllp_free_masks() is already protected by an "if (masks)" before 
the call to the function.

Were you able to reproduce this issue?  If so what is the backtrace?  I 
doubt this patch would fix the issue as it is just double checking the 
variable.

Danny

On 03/26/12 08:59, rod.schu...@bull.com wrote: 
There is no protection on the lllp_free_masks() if the masks pointer is 
NULL. 

The attached patch fixes this problem. 





Reply via email to