php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #68712 suspicious if-else statements
Submitted: 2015-01-02 07:28 UTC Modified: 2016-07-23 17:32 UTC
From: bugreports at internot dot info Assigned: cmb (profile)
Status: Closed Package: GD related
PHP Version: master-Git-2015-01-02 (Git) OS: Linux Ubuntu 14.04
Private report: No CVE-ID: None
View Developer Edit
 [2015-01-02 07:28 UTC] bugreports at internot dot info 
Description:
------------
Hi,

in /ext/gd/libgd/gd_interpolation.c:

1539                        if (m >= src_h - 2) {
1540                                src_offset_x[13] = n;
1541                                src_offset_y[13] = m;
1542                        } else {
1543                                src_offset_x[13] = n;
1544                                src_offset_y[13] = m;
1545                        }


and

1946                                if (m < 1) {
1947                                        src_offset_x[1] = n;
1948                                        src_offset_y[1] = m;
1949                                } else {
1950                                        src_offset_x[1] = n;
1951                                        src_offset_y[1] = m ;
1952                                }



In each, each possiblity is the same. Is this intended?

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2015-01-08 08:44 UTC] [email protected]
-Assigned To: +Assigned To: pajoye
 [2015-07-12 12:56 UTC] [email protected] 
For easier reference:
<https://github.com/php/php-src/blob/php-5.6.11/ext/gd/libgd/gd_interpolation.c#L1539-L1545>
<https://github.com/php/php-src/blob/php-5.6.11/ext/gd/libgd/gd_interpolation.c#L1946-L1952>

It seems to me that the construction of the matrixes in
gdImageScaleBicubicFixed() and gdImageRotateBicubicFixed() need
review. Particularly, setting of index 9 seems to be erroneous:
<https://github.com/php/php-src/blob/php-5.6.11/ext/gd/libgd/gd_interpolation.c#L1507-L1513>
<https://github.com/php/php-src/blob/php-5.6.11/ext/gd/libgd/gd_interpolation.c#L2004-L2010>
 [2016-07-23 17:15 UTC] [email protected]
-Summary: identical code +Summary: superfluous if-else statements
 [2016-07-23 17:32 UTC] [email protected]
-Summary: superfluous if-else statements +Summary: suspicious if-else statements -Status: Assigned +Status: Verified -Assigned To: pajoye +Assigned To: cmb
 [2016-07-23 17:32 UTC] [email protected] 
> In each, each possiblity is the same. Is this intended?

Comparing with external libgd, it is indeed. Therefore the
if-statements are superfluous and should be removed to avoid
confusion.
 [2016-07-23 17:51 UTC] [email protected] 
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=php-src.git;a=commit;h=88838dd28200b53c4da7a558975b195fd41ba446
Log: Fix #68712: suspicious if-else statements
 [2016-07-23 17:51 UTC] [email protected]
-Status: Verified +Status: Closed
 [2016-10-17 10:10 UTC] [email protected] 
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=php-src.git;a=commit;h=88838dd28200b53c4da7a558975b195fd41ba446
Log: Fix #68712: suspicious if-else statements
 
PHP Copyright © 2001-2026 The PHP Group
All rights reserved. 		Last updated: Mon Jan 05 07:00:02 2026 UTC