actually, the whole thing is so chock full of signed vs unsigned 
mixups and the assumption of all the world is 32-bit that i'm not sure 
anymore that correcting them one by one is a good idea.

what would probably work best is that if someone deeply familiar with 
the code (*looks in dan's general direction*) cleaned the data types 
up, then legions of grunts like myself could start cleaning the rest 
with relative confidence...

tree 6aecaae1947b
parent 89bb8da494e7
author Tamas TEVESZ <[email protected]> 1230393784 -3600
committer Tamas TEVESZ <[email protected]> 1230393784 -3600
revision 1629
branch ice

avoid integer overflow
diff --git a/src/wmspec.c b/src/wmspec.c
--- a/src/wmspec.c
+++ b/src/wmspec.c
@@ -393,8 +393,8 @@
 static unsigned long*
 findBestIcon(unsigned long *data, unsigned long items)
 {
-    int size, wanted, d, distance;
-    unsigned long i;
+    int size, wanted, d;
+    unsigned long i, distance;
     unsigned long *icon;
 
     /* better use only 75% of icon_size. For 64x64 this means 48x48
-- 
[-]

mkdir /nonexistent


-- 
To unsubscribe, send mail to [email protected].

Reply via email to