Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stack overflow when passing a URL with spaces #34

Open
GoogleCodeExporter opened this issue May 1, 2015 · 2 comments
Open

Stack overflow when passing a URL with spaces #34

GoogleCodeExporter opened this issue May 1, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

So reproduce, pass the following text into ChangeText.addText(String S)


http://www.liferay.com/web/guest/community/wiki/-/wiki/Main/Development 
Environment 
(Windows)#section-Development+Environment+_28Windows_29-EXTEnvironment

It will infinite loop. 

When I change the URL to:

http://www.liferay.com/web/guest/community/wiki/-/wiki/Main/Development+Environm
ent+(Windows)#section-Development+Environment+_28Windows_29-EXTEnvironment

or:

http://www.liferay.com/web/guest/community/wiki/-/wiki/Main/Development%20Enviro
nment%20(Windows)#section-Development+Environment+_28Windows_29-EXTEnvironment

it seems to be fine. The issue is, this is part of a very very large text file, 
so it's very hard to sanitize it before it gets broken down by your function.

Original issue reported on code.google.com by [email protected] on 12 May 2011 at 11:58

@GoogleCodeExporter
Copy link
Author

Hm, not quite as clear as I would like to be. I'll attach my source and 
compared text to be more clear. Notice the difference is that the original has 
%20s, the changed one doesn't

Original comment by [email protected] on 13 May 2011 at 12:09

Attachments:

@GoogleCodeExporter
Copy link
Author

I am not sure I understand the issue here. Running daisydiff against the 
attached files (original.txt and changed.txt) produces a result (there is no 
infinite loop).
I run 'java -jar daisydiff.jar original.txt changed.txt' and the command 
finishes normally.

Did you mean something else? Am I missing something?

Basically attach some files that can be used directly on the command line tool. 
It is the easiest way to test.

Original comment by [email protected] on 19 May 2011 at 9:12

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

No branches or pull requests

1 participant