On 20 January 2013 21:12, Laurent Vivier <laur...@vivier.eu> wrote: > case GETALL: > case SETALL: > - err = target_to_host_semarray(semid, &array, target_su.array); > + err = target_to_host_semarray(semid, &array, > + tswapal(target_su->array)); > if (err) > return err;
This version is back to the same bug as before where the error case here skips unlocking the struct. PS: please include the version number (v2,v3,etc) in the [PATCH] subject tag so it's easier to distinguish which is the most recent version of a patch when you submit newer versions. thanks -- PMM