From e4f63fba6cfb21688028e2a7607c08018143f791 Mon Sep 17 00:00:00 2001 From: Bill Currie Date: Mon, 30 Jan 2012 09:13:06 +0900 Subject: [PATCH] Fix some integer shift overflows. Thanks to Ozkan Sezer for pointing them out. --- libs/video/renderer/sw/sw_redge.c | 2 +- libs/video/renderer/sw32/sw32_redge.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libs/video/renderer/sw/sw_redge.c b/libs/video/renderer/sw/sw_redge.c index 8f85d9a98..86038b7a2 100644 --- a/libs/video/renderer/sw/sw_redge.c +++ b/libs/video/renderer/sw/sw_redge.c @@ -500,7 +500,7 @@ R_ScanEdges (void) edge_aftertail.prev = &edge_tail; // FIXME: do we need this now that we clamp x in r_draw.c? - edge_sentinel.u = 2000 << 24; // make sure nothing sorts past this + edge_sentinel.u = 32767 << 16; // make sure nothing sorts past this edge_sentinel.prev = &edge_aftertail; // process all scan lines diff --git a/libs/video/renderer/sw32/sw32_redge.c b/libs/video/renderer/sw32/sw32_redge.c index ead186476..4105d3460 100644 --- a/libs/video/renderer/sw32/sw32_redge.c +++ b/libs/video/renderer/sw32/sw32_redge.c @@ -480,7 +480,7 @@ R_ScanEdges (void) edge_aftertail.prev = &edge_tail; // FIXME: do we need this now that we clamp x in r_draw.c? - edge_sentinel.u = 2000 << 24; // make sure nothing sorts past this + edge_sentinel.u = 32767 << 16; // make sure nothing sorts past this edge_sentinel.prev = &edge_aftertail; // process all scan lines