php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #61979 LibXMLError Object property 'message' contains LF at the end of string
Submitted: 2012-05-09 09:23 UTC Modified: 2021-03-16 16:04 UTC
From: piotr at smartwebstudio dot cz Assigned: cmb (profile)
Status: Not a bug Package: *XML functions
PHP Version: 5.3.13 OS: Windows Vista 32bit
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: piotr at smartwebstudio dot cz
New email:
PHP Version: OS:

 

 [2012-05-09 09:23 UTC] piotr at smartwebstudio dot cz
Description:
------------
---
From manual page: http://www.php.net/manual/pl/function.libxml-get-last-error.php
---
Tested on PHP 5.3.0, Apache 2.2.11

LibXMLError Object property 'message' contains LF (ASCII 10) at the end of 
string.
Makes it harder to compare error messages that should be ignored or assign 
translation strings.

Test script:
---------------
libxml_use_internal_errors(true);
// Define xml string that contains known error
$vcard = new SimpleXMLElement("<vCard xmlns='vcard-temp'></vcard>");

// Retrieve last LibXMLError object
$last_error = libxml_get_last_error();

// Should be true
var_dump($last_error->message == "xmlns: URI vcard-temp is not absolute");

// Workaround
var_dump(trim($last_error->message) == "xmlns: URI vcard-temp is not absolute");

Expected result:
----------------
bool(true)
bool(true)

Actual result:
--------------
bool(false)
bool(true)

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-06-13 18:14 UTC] cmb@php.net
In my opinion this is not a bug, as I understand the error message
as "human-readable informative error message"[1]. And I would
rather check for the error code to avoid issues due to libxml
changing the error messages or localization issues.

[1] <http://www.xmlsoft.org/html/libxml-xmlerror.html#xmlError>
 [2018-03-14 23:10 UTC] cmb@php.net
-Package: XML related +Package: *XML functions
 [2021-03-16 16:04 UTC] cmb@php.net
-Status: Open +Status: Not a bug -Assigned To: +Assigned To: cmb
 [2021-03-16 16:04 UTC] cmb@php.net
Well, the LFs are indeed provided by libxml2 already[1], and it
makes no sense to strip them out.

[1] <https://github.com/GNOME/libxml2/blob/868e49cffdbfddeab0d1c391e0ce7f4f98ea59a1/parser.c#L9289>
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Oct 08 22:01:27 2024 UTC