Lei Zhang : gdiplus: Do not access freed memory in a test.
Alexandre Julliard
julliard at winehq.org
Tue Apr 22 06:47:28 CDT 2008
Module: wine
Branch: master
Commit: 321dfcc7a987d0be5738bdc8642d0531fb6db7d3
URL: http://source.winehq.org/git/wine.git/?a=commit;h=321dfcc7a987d0be5738bdc8642d0531fb6db7d3
Author: Lei Zhang <thestig at google.com>
Date: Mon Apr 21 16:15:58 2008 -0700
gdiplus: Do not access freed memory in a test.
---
dlls/gdiplus/tests/graphics.c | 20 +++++++++++++++-----
1 files changed, 15 insertions(+), 5 deletions(-)
diff --git a/dlls/gdiplus/tests/graphics.c b/dlls/gdiplus/tests/graphics.c
index edd98d7..77f1597 100644
--- a/dlls/gdiplus/tests/graphics.c
+++ b/dlls/gdiplus/tests/graphics.c
@@ -72,20 +72,30 @@ static void check_no_duplicates(node * log)
{
INT dups = 0;
node * temp = NULL;
+ node * temp2 = NULL;
+ node * orig = log;
if(!log)
goto end;
do{
- HeapFree(GetProcessHeap(), 0, temp);
temp = log;
- while((temp = temp->next))
- if(log->data == temp->data)
+ while((temp = temp->next)){
+ if(log->data == temp->data){
dups++;
-
+ break;
+ }
+ if(dups > 0)
+ break;
+ }
}while((log = log->next));
- HeapFree(GetProcessHeap(), 0, temp);
+ temp = orig;
+ do{
+ temp2 = temp->next;
+ HeapFree(GetProcessHeap(), 0, temp);
+ temp = temp2;
+ }while(temp);
end:
expect(0, dups);
More information about the wine-cvs
mailing list