1 files changed, 9 insertions, 10 deletions

diff --git a/src/screen.c b/src/screen.c
index c41234f..ab912a1 100644
--- a/src/screen.c
+++ b/src/screen.c
@@ -808,12 +808,11 @@ void term_screen_set_alternate(struct term *t, bool enabled)
     }
 }
 
+/* The caller has checked that requested is not negative. */
 static int scrollback_lines(int requested)
 {
     if (requested == 0)
         return TERM_SCROLLBACK_DEFAULT;
-    if (requested < 0)
-        return 0;
     return requested > 1000000 ? 1000000 : requested;
 }
 
@@ -850,10 +849,12 @@ struct term *term_new(const struct term_options *opt)
     for (size_t i = 0; i < sizeof(opt->reserved) / sizeof(opt->reserved[0]); i++)
         if (opt->reserved[i])
             return NULL;
-    int cols = opt->size.x < 1 ? 80 : opt->size.x;
-    int rows = opt->size.y < 1 ? 24 : opt->size.y;
-    if (cols > TERM_MAX_COLS)
-        cols = TERM_MAX_COLS;
+    if (opt->size.x < 0 || opt->size.y < 0 || opt->size.x > TERM_MAX_COLS)
+        return NULL;
+    if (opt->scrollback < 0 || (unsigned)opt->cursor_style > TERM_CURSOR_BAR)
+        return NULL;
+    int cols = opt->size.x == 0 ? 80 : opt->size.x;
+    int rows = opt->size.y == 0 ? 24 : opt->size.y;
 
     static _Atomic uint64_t next_serial = 1;
     struct term *t = calloc(1, sizeof(*t));
@@ -898,7 +899,7 @@ struct term *term_new(const struct term_options *opt)
     t->cur_fg = TERM_COLOR_FG;
     t->cur_bg = TERM_COLOR_BG;
     t->cursor_visible = true;
-    t->default_cursor_style = opt->cursor_style > TERM_CURSOR_BAR ? TERM_CURSOR_BLOCK : opt->cursor_style;
+    t->default_cursor_style = opt->cursor_style;
     t->default_cursor_blink = !opt->cursor_steady;
     t->cursor_style = t->default_cursor_style;
     t->cursor_blink = t->default_cursor_blink;
@@ -1526,10 +1527,8 @@ bool term_resize(struct term *t, struct term_vector2i size)
         return false;
     int cols = size.x;
     int rows = size.y;
-    if (cols < 1 || rows < 1)
+    if (cols < 1 || rows < 1 || cols > TERM_MAX_COLS)
         return false;
-    if (cols > TERM_MAX_COLS)
-        cols = TERM_MAX_COLS;
     term_write_begin(t);
     if (cols == t->cols && rows == t->rows) {
         term_write_end(t);