[cairo-commit] pixman/src
Carl Worth
cworth at kemper.freedesktop.org
Mon Mar 13 13:22:36 PST 2006
pixman/src/fbpict.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
New commits:
diff-tree 5d89faf6b37bb0367c8ba8f8637fc80bd745c888 (from 6f0d8eedc504e8af2224c502be25dc82b5df70f4)
Author: Carl Worth <cworth at cworth.org>
Date: Mon Mar 13 13:20:29 2006 -0800
fbCompositeTrans_0888xnx0888: Fix suspicious logical negation of pointer value.
Surely, the ! instead of ~ here was just a typo. I'd feel better if I
had a failing test case for this though, (time to break out gcov
again).
Fixes bug #6185:
Strange pointer operations in fbCompositeTrans_0888xnx0888 (pixman/src/fbpict.c)
https://bugs.freedesktop.org/show_bug.cgi?id=6185
diff --git a/pixman/src/fbpict.c b/pixman/src/fbpict.c
index e074174..ef7f4e2 100644
--- a/pixman/src/fbpict.c
+++ b/pixman/src/fbpict.c
@@ -1151,7 +1151,7 @@ fbCompositeTrans_0888xnx0888(pixman_oper
setupPackedReader(ws,wt,isrc,wsrc,workingSource);
/* get to word aligned */
- switch(!(long)dst&3)
+ switch(~(long)dst&3)
{
case 1:
readPackedSource(rs);
@@ -1227,7 +1227,7 @@ fbCompositeTrans_0888xnx0888(pixman_oper
srcLine += srcStride;
w = width*3;
/* get to word aligned */
- switch(!(long)src&3)
+ switch(~(long)src&3)
{
case 1:
rd=alphamaskCombine24(*src++, *dst)>>8;
More information about the cairo-commit
mailing list