[Mesa-dev] [PATCH] mesa: Fix wrong sizeof argument in _math_matrix_copy.
Brian Paul
brian.e.paul at gmail.com
Mon Jul 30 07:17:03 PDT 2012
On Mon, Jul 30, 2012 at 7:59 AM, Brian Paul <brian.e.paul at gmail.com> wrote:
> On Sun, Jul 29, 2012 at 6:54 PM, Vinson Lee <vlee at freedesktop.org> wrote:
>> Fixes Coverity wrong sizeof argument defect.
>>
>> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
>> ---
>> src/mesa/math/m_matrix.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/mesa/math/m_matrix.c b/src/mesa/math/m_matrix.c
>> index 40f9229..7c4e9fd 100644
>> --- a/src/mesa/math/m_matrix.c
>> +++ b/src/mesa/math/m_matrix.c
>> @@ -1437,7 +1437,7 @@ void
>> _math_matrix_copy( GLmatrix *to, const GLmatrix *from )
>> {
>> memcpy( to->m, from->m, sizeof(Identity) );
>> - memcpy(to->inv, from->inv, sizeof(from->inv));
>> + memcpy(to->inv, from->inv, sizeof(*from->inv));
>> to->flags = from->flags;
>> to->type = from->type;
>> }
Vinson, I've pushed your patch since there was already a bug report about this.
> It would probably be good to update the preceding line as well. It's
> kind of weird to use sizeof(X) when X isn't the source or dest to the
> memcpy().
I'll post a patch to fix this and clean up a few other cases.
-Brian
More information about the mesa-dev
mailing list