Skip to content

Conversation

filippo-biondi
Copy link
Contributor

Since the end of boundary patches in binary file was checked by stripping the line and checking if the result was equal to '}' it was possible that the end of a patch was miss-detected, since a '}' surrounded by characters that are removed by the strip function could appear randomly in the binary content (and indeed happened in my case ).

I updated the check to remove only withe spaces, tabs and new lines in the stripping operation and check for the ');' characters at the end of the previous line to determine if the actual end of the boundary patch is reached.

I think its theoretically possible that the end of a patch is still miss-detected (since a ');' followed on the next line by a '}' surrounded by tabs or withe-spaces could still appear randomly in the binary content) but it's much more unlikely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant