From 00ab71f3d33b0794cee87bc5a0162429577c0571 Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Fri, 27 Oct 2023 12:50:58 -0700 Subject: [PATCH] Remove 1986/hague from bugs.md --- bugs.md | 28 ---------------------------- 1 file changed, 28 deletions(-) diff --git a/bugs.md b/bugs.md index d5cfa9e12..a742c1a54 100644 --- a/bugs.md +++ b/bugs.md @@ -477,34 +477,6 @@ emulator to test it) but running the code on the binary itself produces a This entry is known to segfault after printing its output. It was documented by the judges and shouldn't be fixed. -## [1986/hague](1986/hague/hague.c) ([README.md](1986/hague/README.md)) -### STATUS: uses gets() - change to fgets() if possible - -This entry uses `gets()` which is unsafe. In particular the buffer size is a -mere 81 and it does not read a single string at command invocation but instead -until the program is terminated. This size can be increased to start out but it -would be better if it used `fgets()`. - -The way this entry is formed has made it not as simple as some of the others -that have been addressed at least in part. It is easy to get it to use `fgets()` -and this has been done in the [alternate code](1986/hague/hague.alt.c) but it -will segfault after outputting the (correct) text in one of the forms. In -particular whereas this: - -```sh -./hague.alt -foo - -works the following will print out the correct output but then segfault: - -``` -```sh -echo foo | ./hague.alt -``` - -Whether or not it is worth making the entry itself (since the output is correct) -this alt version is to be determined later but it's might very well not be. - # 1987