NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: bin/55531: [PATCH] script(1): Check if stdin is a terminal + other error handling



Hypothesis: reintroduction of the extra termreset() makes it necessary
to have "isterm = 0" at the beginning of "playback()" so that if a
malformed recording file forces case 'e' before case 's', then
tcsetattr will not be called by termreset(). In that case, an updated
patch follows. Also, this patch removes an extra whitespace after
err(1, "fstat failed").

On 8/2/20, Soumendra Ganguly <soumendra%tamu.edu@localhost> wrote:
> The following reply was made to PR bin/55531; it has been noted by GNATS.
>
> From: Soumendra Ganguly <soumendra%tamu.edu@localhost>
> To: gnats-bugs%netbsd.org@localhost
> Cc:
> Subject: Re: bin/55531: [PATCH] script(1): Check if stdin is a terminal +
>  other error handling
> Date: Sun, 2 Aug 2020 03:44:03 -0500
>
>  This follows #55529
>
>  On 8/2/20, gnats-admin%netbsd.org@localhost <gnats-admin%netbsd.org@localhost> wrote:
>  > Thank you very much for your problem report.
>  > It has the internal identification `bin/55531'.
>  > The individual assigned to look at your
>  > report is: bin-bug-people.
>  >
>  >>Category:       bin
>  >>Responsible:    bin-bug-people
>  >>Synopsis:       [PATCH] script(1): Check if stdin is a terminal + other
>  >> error handling
>  >>Arrival-Date:   Sun Aug 02 08:30:00 +0000 2020
>  >
>  >
>
>



Home | Main Index | Thread Index | Old Index