HtmlCleaner / Bugs / #211 IOException when trying to reset unsuccessful CDATA end lookup

#211 IOException when trying to reset unsuccessful CDATA end lookup

Milestone: v2.23

Status: closed-fixed

Owner: nobody

Labels: None

Priority: 5

Updated: 2019-09-04

Created: 2018-09-17

Creator: Anthony Pessy

Private: No

Reproduced with the following page:

http://www.okaidi.fr/vetements-fille/?produits-par-page=1500

The _reader.reset() call in HtmlTokenizer.containsEndCData will produce a IOException with Mark invalid because we read too much from the input.

Reading one less char (read < MAX_BUFFER_SIZE - 1) will solve the issue.

Discussion

Scott Wilson - 2018-09-17

status: open --> open-accepted

Group: v 2.7 --> v2.23
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Scott Wilson - 2018-09-17

Thanks Anthony, I'll look into it!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Scott Wilson - 2019-09-04

status: open-accepted --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Scott Wilson - 2019-09-04

Good spot there - fixed in 2.23

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

IOException when trying to reset unsuccessful CDATA end lookup

Group

Searches

Help

#211 IOException when trying to reset unsuccessful CDATA end lookup

Discussion