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

Reply via email to