On 02/10/2015 14:48, Laurent Vivier wrote:
> +        target_fd_trans = g_realloc(target_fd_trans,
> +                                    target_fd_max * sizeof(TargetFdTrans));

This should be TargetFdTrans * (reported by Coverity).  Even better you
could use g_renew.

It's harmless because sizeof(TargetFdTrans) > sizeof(TargetFdTrans *),
but it should be fixed nevertheless.

Paolo

> +        memset((void *)(target_fd_trans + oldmax), 0,
> +               (target_fd_max - oldmax) * sizeof(TargetFdTrans *));

Reply via email to