WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#14187 closed defect (bug) (fixed)

strrpos supports only a single character for the needle in PHP4

Reported by: mailnew2ster Owned by:
Priority: normal Milestone: 3.1
Component: HTTP API Version:
Severity: normal Keywords: has-patch needs-testing
Cc: Focuses:

Description

class-http.php, line 1411:
​http://core.trac.wordpress.org/browser/trunk/wp-includes/class-http.php#L1411

strrpos is used, while it does not work as expected on PHP4. Can be replaced by strpos, as the position is not needed in that case.

Attachments (1)

.diff​ (450 bytes ) - added by mailnew2ster 16 years ago.

Download all attachments as: .zip

Change History (4)

@mailnew2ster
16 years ago

#1 @dd32
16 years ago

  • Keywords has-patch needs-testing added

From what i can see, Changing from looking for the last occurrence, to the first occurrence as in the patch wouldnt cause an issue..

#2 @nacin
16 years ago

  • Milestone Awaiting Review → 3.1

#3 @nacin
16 years ago

  • Resolution → fixed
  • Status new → closed

(In [16762]) We only need strpos here. Ensures PHP4 compat. props mailnew2ster, fixes #14187.

Note: See TracTickets for help on using tickets.