php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #13995 continuation of bug #13866: strtok() misparses empty fields
Submitted: 2001-11-08 20:29 UTC Modified: 2001-11-09 02:12 UTC
From: rcpj at panix dot com Assigned:
Status: Closed Package: Strings related
PHP Version: 4.0.6 OS: OpenBSD 2.9-stable
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: rcpj at panix dot com
New email:
PHP Version: OS:

 

 [2001-11-08 20:29 UTC] rcpj at panix dot com
If a string contains two delimiters side by side (i.e. an empty field), strtok() treats the double delimiter as a single one and the results are shifted into the wrong variable. The previous version of the bug kept the second delimiter in the value returned by the function. This version does not, but it returns the wrong substring.

PHP version is 4.2.0-dev

(how do I report the correct version on the report form? Shouldn't there be at least a "other"?)

Example of the bug:
  
$String= "field1|field2||field4";

$Item1=strtok($String,"|");    // $Item1: field1
$Item2=strtok("|");               // $Item2: field2
$Item3=strtok("|");                // $Item3: field4
$Item4=strtok("|");                // $Item4 is empty

compare the outputs of:

http://www.pierrejelenc.com/PHP/test-module.php    (correct)
and
http://www.web-ho.com/PHP/test-module.php    (incorrect)

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2001-11-09 02:12 UTC] derick@php.net
This is the expected behavior. It conforms to alteast the C-libraries on Linux, Win32 and Solaris.

Derick
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Mar 29 07:01:28 2024 UTC