andreacavalli/xserver-multidpi
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

ddc: Fix memory leak in GetEDID_DDC1

Browse Source 
Mark argument to DDC_checksum as const too.

Signed-off-by: Matt Turner <mattst88@gmail.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Keith Packard <keithp@keithp.com>
This commit is contained in:
Matt Turner 2010-07-27 23:32:36 -04:00 committed by Keith Packard
parent 750d4e82a0
commit 7e0575baf1
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hw/xfree86/ddc/ddc.c
View File

@ -102,7 +102,7 @@ resort(unsigned char *s_block)
}
static int
DDC_checksum(unsigned char *block, int len)
DDC_checksum(const unsigned char *block, int len)
{
int i, result = 0;
int not_null = 0;
@ -149,7 +149,10 @@ GetEDID_DDC1(unsigned int *s_ptr)
d_pos++;
}
free(s_ptr);
if (d_block && DDC_checksum(d_block,EDID1_LEN)) return NULL;
if (d_block && DDC_checksum(d_block,EDID1_LEN)) {
free(d_block);
return NULL;
}
return (resort(d_block));
}