From 6e7996f1a84793a3d88e141a340b74fdae8dbf73 Mon Sep 17 00:00:00 2001 From: nick black Date: Sun, 7 Jun 2020 16:11:21 -0400 Subject: [PATCH] notcurses_init: properly check flags for unknowns --- src/lib/notcurses.c | 2 +- src/poc/blitters.c | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/lib/notcurses.c b/src/lib/notcurses.c index 3b8fb247c..3c13db3f2 100644 --- a/src/lib/notcurses.c +++ b/src/lib/notcurses.c @@ -735,7 +735,7 @@ notcurses* notcurses_init(const notcurses_options* opts, FILE* outfp){ fprintf(stderr, "Provided an illegal negative margin, refusing to start\n"); return NULL; } - if(opts->flags > NCOPTION_NO_ALTERNATE_SCREEN){ + if(opts->flags > (NCOPTION_NO_ALTERNATE_SCREEN << 1u)){ fprintf(stderr, "Provided an illegal Notcurses option, refusing to start\n"); return NULL; } diff --git a/src/poc/blitters.c b/src/poc/blitters.c index ba7489ec4..6b2b07fc6 100644 --- a/src/poc/blitters.c +++ b/src/poc/blitters.c @@ -16,6 +16,9 @@ int main(int argc, char** argv){ .flags = NCOPTION_INHIBIT_SETLOCALE | NCOPTION_NO_ALTERNATE_SCREEN, }; struct notcurses* nc = notcurses_init(&nopts, NULL); + if(nc == NULL){ + return EXIT_FAILURE; + } struct ncplane* std = notcurses_stdplane(nc); // FIXME not all work yet; this subset does const int blitters[] = {