From e2d7d06f147ee15429053fd40ade631ac08d7e23 Mon Sep 17 00:00:00 2001 From: Christian Heller Date: Tue, 27 Aug 2013 02:39:15 +0200 Subject: [PATCH] Uise uint32_t to test for a value going beyond UINT16_MAX. --- src/windows.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/windows.c b/src/windows.c index a7bf88f..6257d4b 100644 --- a/src/windows.c +++ b/src/windows.c @@ -365,23 +365,23 @@ static void shift_win_backward(struct WinMeta * wmeta) extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta * wmeta) { wmeta->_screen = screen; - wmeta->padframe.size.y = getmaxy(screen); - wmeta->padframe.size.x = getmaxx(screen); - if ( wmeta->padframe.size.y > UINT16_MAX - || wmeta->padframe.size.x > UINT16_MAX) + uint32_t maxy_test = getmaxy(screen); + uint32_t maxx_test = getmaxx(screen); + if (maxy_test > UINT16_MAX || maxx_test > UINT16_MAX) { return 2; } + wmeta->padframe.size.y = maxy_test; + wmeta->padframe.size.x = maxx_test; wmeta->_chain_start = 0; wmeta->_chain_end = 0; wmeta->pad_offset = 0; - WINDOW * test; - test = newpad(wmeta->padframe.size.y, 1); - if (NULL == test) + WINDOW * pad_test = newpad(wmeta->padframe.size.y, 1); + if (NULL == pad_test) { return 1; } - wmeta->padframe.curses_win = test; + wmeta->padframe.curses_win = pad_test; wmeta->active = 0; return 0; } -- 2.30.2