bug#56048: closed ([PATCH] bindat (strz): Null terminate fixed-length strings if there is room)

[GNU bug Tracking System]

Your message dated Wed, 22 Jun 2022 16:57:19 +0300
with message-id <835ykskdhc.fsf@gnu.org>
and subject line Re: bug#56048: [PATCH] bindat (strz): Null terminate 
fixed-length strings if there is room
has caused the debbugs.gnu.org bug report #56048,
regarding [PATCH] bindat (strz): Null terminate fixed-length strings if there 
is room
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
56048: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=56048
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] bindat (strz): Null terminate fixed-length strings if there is room Date: Fri, 17 Jun 2022 18:52:51 -0400 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1

X-Debbugs-CC: monnier@iro.umontreal.ca

Two patches attached:

Patch 1:

    ; bindat (strz): Move all pack logic to pack function

Patch 2:

    bindat (strz): Null terminate fixed-length strings if there is room

    * lisp/emacs-lisp/bindat.el (bindat--pack-strz): For fixed-length strz
    fields, explicitly write a null terminator after the packed string if
    there is room.
    * doc/lispref/processes.texi (Bindat Types): Update documentation.
    * test/lisp/emacs-lisp/bindat-tests.el (bindat-test--str-strz-prealloc):
    Update tests.

0001-bindat-strz-Move-all-pack-logic-to-pack-function.patch
Description: Text Data

0002-bindat-strz-Null-terminate-fixed-length-strings-if-t.patch
Description: Text Data

OpenPGP_signature
Description: OpenPGP digital signature

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#56048: [PATCH] bindat (strz): Null terminate fixed-length strings if there is room
>
>
>
>
> Date: 
>
> Wed, 22 Jun 2022 16:57:19 +0300
>
>
>
>
> > Cc: 56048@debbugs.gnu.org
> > Date: Tue, 21 Jun 2022 16:44:06 -0400
> > From:  Stefan Monnier via "Bug reports for GNU Emacs,
> >  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
> > 
> > Thanks, Richard.  Looks good to me.
>
> Installed on master.
>
>
> --- End Message ---