Re: telnet: Handle integer overflow gracefully.

Simon Josefsson via Bug reports for the GNU Internet utilities <[email protected]> Sun, 25 Aug 2024 16:37:26 +0200
Newsgroups gmane.comp.gnu.inetutils.bugs
Message-ID <[email protected]>
Erik Auerswald <[email protected]> writes:

> Hi,
>
> On Fri, Aug 23, 2024 at 10:56:30PM -0700, Collin Funk wrote:
>> 
>> I've pushed the attached patch removing an integer overflow from telnet.
>
> Thanks!  Should this be added to NEWS?

Thanks Collin, and yes please :)

>> The overflow occurs went sending 'send dont <value>' but the value
>> exceeds INT_MAX.
>
> 'send dont <value>' is a "hidden" command, i.e., it is not shown in the
> output of 'send ?', because the 'help' string is NULL (see the definition
> of Sendlist[] in telnet/commands.c).  Does someone know the reason to
> hide this command (and similar ones) from the online help?

I don't know the history -- but (sounding like a broken record by now)
did anyone check *BSD telnet behaviour?  I'm not a fan of intentionally
not documentating supported commands, so +1 on any patch to document
this from me, regardless of what *BSD telnet does.

/Simon
signature.asc (application/pgp-signature, 255 B)
-----BEGIN PGP SIGNATURE-----

iIoEARYIADIWIQSjzJyHC50xCrrUzy9RcisI/kdFogUCZstBphQcc2ltb25Aam9z
ZWZzc29uLm9yZwAKCRBRcisI/kdFooT5AP4zcwKNDFYj0PkCrjztQAcvg3ICqqm5
z1+B9lambwkA1gEA7rvGMVoXyBe/mdH8Cj89Je6CIvkykZIsVg93+cglsAY=
=+j23
-----END PGP SIGNATURE-----