Added assert()s to check various input parameters to core functions

Bill-Gray · Oct 3, 2024 · 027c3a5 · 027c3a5
1 parent d2f479a
commit 027c3a5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/pdcurses/insch.c b/pdcurses/insch.c
@@ -79,6 +79,7 @@ int winsch(WINDOW *win, chtype ch)
     x = win->_curx;
     y = win->_cury;
 
+    assert( y <= win->_maxy && x <= win->_maxx && y >= 0 && x >= 0);
     if (y > win->_maxy || x > win->_maxx || y < 0 || x < 0)
         return ERR;
 

diff --git a/pdcurses/touch.c b/pdcurses/touch.c
@@ -65,6 +65,9 @@ void PDC_set_changed_cells_range( WINDOW *win, const int y, const int start, con
 {
     assert( win);
     assert( y >= 0 && y < win->_maxy);
+    assert( start >= 0 || start == _NO_CHANGE);
+    assert( start <= end);
+    assert( end < win->_maxx);
     win->_firstch[y] = start;
     win->_lastch[y] = end;
 }
@@ -81,6 +84,9 @@ void PDC_mark_cells_as_changed( WINDOW *win, const int y, const int start, const
 {
     assert( win);
     assert( y >= 0 && y < win->_maxy);
+    assert( start >= 0 || start == _NO_CHANGE);
+    assert( start <= end);
+    assert( end < win->_maxx);
     if( win->_firstch[y] == _NO_CHANGE)
     {
         win->_firstch[y] = start;

diff --git a/pdcurses/window.c b/pdcurses/window.c
@@ -203,6 +203,8 @@ WINDOW *PDC_makelines(WINDOW *win)
     assert( win);
     if (!win)
         return (WINDOW *)NULL;
+    assert( win->_maxy > 0);
+    assert( win->_maxx > 0);
 
     nlines = win->_maxy;
     ncols = win->_maxx;