Bug #63228 [Com]: -Werror=format-security error in lsapi code

From: Date: Sat, 11 Oct 2014 06:38:44 +0000
Subject: Bug #63228 [Com]: -Werror=format-security error in lsapi code
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-188018@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=63228&edit=1

 ID:                 63228
 Comment by:         wofaday at qq dot com
 Reported by:        glen at delfi dot ee
 Summary:            -Werror=format-security error in lsapi code
 Status:             Closed
 Type:               Bug
 Package:            Compile Failure
 PHP Version:        5.4.7, 5.6, master
 Assigned To:        gwang
 Block user comment: N
 Private report:     N

 New Comment:

this is a bug for php 5.6.1,why no body pay close attention to this ,and nobody to  modify to this ?
if you use php 5.6.1 make,it will show :
make: *** [sapi/litespeed/lsapi_main.lo] Error 1

but php 5.6.0 is ok


Previous Comments:
------------------------------------------------------------------------
[2014-10-07 23:31:39] stas@php.net

Automatic comment on behalf of gwang
Revision: http://git.php.net/?p=php-src-security.git;a=commit;h=9c52d09ebc62683f26338123bcda8068f162541d
Log: sapi/litespeed/lsapi_main.c: Fix bug #63228

------------------------------------------------------------------------
[2014-10-07 23:20:39] stas@php.net

Automatic comment on behalf of gwang
Revision: http://git.php.net/?p=php-src-security.git;a=commit;h=9c52d09ebc62683f26338123bcda8068f162541d
Log: sapi/litespeed/lsapi_main.c: Fix bug #63228

------------------------------------------------------------------------
[2014-05-02 20:28:22] tyrael@php.net

it is as Adam guessed, Felipe cherry-picked/merged this commit into 5.6 and master but forgot to add
a NEWS entry, and when I noticed, I used the commiter name without looking more into it.
sorry for the inconvenience, and thanks for the report and the patch!

------------------------------------------------------------------------
[2014-05-02 17:27:52] aharvey@php.net

I suspect it happened by accident: I think the NEWS was added later by someone else and they
probably just worked off the committer name.

I've fixed it up (on all branches) to include you. Sorry about that!

------------------------------------------------------------------------
[2014-05-02 08:18:56] glen at delfi dot ee

btw, why in NEWS is "George Wang" name not "Elan Ruusamäe" ? I reported and
fixed it (provided patch)

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=63228


--
Edit this bug report at https://bugs.php.net/bug.php?id=63228&edit=1


Thread (20 messages)

« previous php.bugs (#188018) next »