Re: lib/52849: libedit core dumps on saving history file with psql from postgresql*-client packages on amd64 platform

[Paul J Lavoie <plavoie%hvc.rr.com@localhost>]

The following reply was made to PR lib/52849; it has been noted by GNATS.

From: Paul J Lavoie <plavoie%hvc.rr.com@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: lib-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost,
 netbsd-bugs%netbsd.org@localhost
Subject: Re: lib/52849: libedit core dumps on saving history file with psql
 from postgresql*-client packages on amd64 platform
Date: Sat, 23 Dec 2017 20:15:01 -0500

 The core dump issue has been fixed by this change.
 
 The odd issue with the multi-line copy/paste still remains, though. Not =
 sure how to collect more info on that one.
 
 > On Dec 23, 2017, at 1:30 PM, Valery Ushakov <uwe%stderr.spb.ru@localhost> wrote:
 >=20
 > The following reply was made to PR lib/52849; it has been noted by =
 GNATS.
 >=20
 > From: Valery Ushakov <uwe%stderr.spb.ru@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc:=20
 > Subject: Re: lib/52849: libedit core dumps on saving history file with =
 psql
 > from postgresql*-client packages on amd64 platform
 > Date: Sat, 23 Dec 2017 21:28:13 +0300
 >=20
 > On Sat, Dec 23, 2017 at 21:04:26 +0300, Valery Ushakov wrote:
 >=20
 >> =46rom a quick look 1.58 of history.c (from Sep 1) looks highly
 >> suspicious as it introduced a call like=20
 >>=20
 >>  history_save_fp(h, va_arg(va, size_t), va_arg(va, FILE *));
 >>=20
 >> and of course C99 even cites that as an example if UB:
 >=20
 > I've committed a fix in history.c revision 1.59.  Please, can you
 > update and test?  TIA!
 >=20
 > -uwe
 >=20