commit ab6f1aeac761178fb70863681eab9e681aa3f379 from: Neven Sajko via: David du Colombier <0intro@gmail.com> date: Thu Jan 02 15:16:41 2020 UTC cb: fix a null pointer dereference Just added a pair of parentheses. I also ran cb on cb.c to beautify the code. This is actually on Gerrit from 2016: https://plan9port-review.googlesource.com/c/plan9/+/1574 Change-Id: I5e234adba0f95c13d6eecb121bf11bba4bf54566 commit - a1ead676e69d46df89dd0de12cf86380c5773353 commit + ab6f1aeac761178fb70863681eab9e681aa3f379 blob - 0b2458965d54aae9186f44a406c58dd85d85eaaa blob + 9e611f042c75f49897d80d1c4f47acb1ba1a1848 --- src/cmd/cb/cb.c +++ src/cmd/cb/cb.c @@ -29,7 +29,7 @@ main(int argc, char *argv[]) maxleng -= (maxleng + 5)/10; continue; default: - fprint(2, "cb: illegal option %c\n", *argv[1]); + fprint(2, "cb: illegal option %c\n", (*argv)[1]); exits("boom"); } } @@ -354,7 +354,7 @@ work(void){ continue; } } - else if (lbegin == 0 || p > string) + else if (lbegin == 0 || p > string) if(strict) putch(c,NO); else putch(c,YES); @@ -969,7 +969,7 @@ cont: if(nlct++ > 2)goto done; } puttmp(c,1); - star: +star: if(puttmp((c=Bgetc(input)),1) == '/'){ beg = tp; puttmp((c=Bgetc(input)),1);