unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Backport fix for Bug #19213/#23955
       [not found] ` <CAHTjjFLD=qXX=3_du8jjFpk9PvZZgGvVd2ewx2mGi6pKevKsiQ@mail.gmail.com>
@ 2016-07-13 23:06   ` npostavs
  2016-07-14  0:33     ` John Wiegley
  0 siblings, 1 reply; 4+ messages in thread
From: npostavs @ 2016-07-13 23:06 UTC (permalink / raw)
  To: emacs-devel; +Cc: Gustavo Tavares Cabral

This bug was fixed replacing a memcpy with a memmove, which is safe by
definition, so I think we should backport it to emacs-25.

354f9f0f "Use memmove instead of memcpy on overlapping regions"

http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19213
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=23955



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Backport fix for Bug #19213/#23955
  2016-07-13 23:06   ` Backport fix for Bug #19213/#23955 npostavs
@ 2016-07-14  0:33     ` John Wiegley
  2016-07-14 15:02       ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: John Wiegley @ 2016-07-14  0:33 UTC (permalink / raw)
  To: npostavs; +Cc: Gustavo Tavares Cabral, emacs-devel

>>>>> "n" == npostavs  <npostavs@users.sourceforge.net> writes:

n> This bug was fixed replacing a memcpy with a memmove, which is safe by
n> definition, so I think we should backport it to emacs-25.

Looks safe to me.

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Backport fix for Bug #19213/#23955
  2016-07-14  0:33     ` John Wiegley
@ 2016-07-14 15:02       ` Eli Zaretskii
  2016-07-15  0:40         ` Noam Postavsky
  0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2016-07-14 15:02 UTC (permalink / raw)
  To: John Wiegley; +Cc: emacs-devel, gustavotcabral, npostavs

> From: John Wiegley <jwiegley@gmail.com>
> Date: Wed, 13 Jul 2016 17:33:11 -0700
> Cc: Gustavo Tavares Cabral <gustavotcabral@gmail.com>, emacs-devel@gnu.org
> 
> >>>>> "n" == npostavs  <npostavs@users.sourceforge.net> writes:
> 
> n> This bug was fixed replacing a memcpy with a memmove, which is safe by
> n> definition, so I think we should backport it to emacs-25.
> 
> Looks safe to me.

Agreed.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Backport fix for Bug #19213/#23955
  2016-07-14 15:02       ` Eli Zaretskii
@ 2016-07-15  0:40         ` Noam Postavsky
  0 siblings, 0 replies; 4+ messages in thread
From: Noam Postavsky @ 2016-07-15  0:40 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: John Wiegley, Gustavo Tavares Cabral, Emacs developers

unarchive 19213
fixed 19213 25.1
forcemerge 19213 23955
quit


On Thu, Jul 14, 2016 at 11:02 AM, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: John Wiegley <jwiegley@gmail.com>
>> Date: Wed, 13 Jul 2016 17:33:11 -0700
>> Cc: Gustavo Tavares Cabral <gustavotcabral@gmail.com>, emacs-devel@gnu.org
>>
>> >>>>> "n" == npostavs  <npostavs@users.sourceforge.net> writes:
>>
>> n> This bug was fixed replacing a memcpy with a memmove, which is safe by
>> n> definition, so I think we should backport it to emacs-25.
>>
>> Looks safe to me.
>
> Agreed.

I've pushed to emacs-25 (916abe91fd).



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-07-15  0:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <87a8hn2g0e.fsf@gustavo-VirtualBox.i-did-not-set--mail-host-address--so-tickle-me>
     [not found] ` <CAHTjjFLD=qXX=3_du8jjFpk9PvZZgGvVd2ewx2mGi6pKevKsiQ@mail.gmail.com>
2016-07-13 23:06   ` Backport fix for Bug #19213/#23955 npostavs
2016-07-14  0:33     ` John Wiegley
2016-07-14 15:02       ` Eli Zaretskii
2016-07-15  0:40         ` Noam Postavsky

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).