Empty URI port crash fix.

byorgey / haxr

Other

10 stars 16 forks source link

Empty URI port crash fix. #9

Closed akru closed 8 years ago

akru commented 8 years ago

Fix for the issue https://github.com/byorgey/BlogLiterately/issues/22#issuecomment-123163552.

Added:

safe tail for the URI port grab;

byorgey commented 8 years ago

Thanks for the quick fix! I just have a few small comments. I'm kicking myself for not questioning the use of tail in the previous patch. =)

akru commented 8 years ago

Thank you! drop 1 is really simple! Could I create new PR with alternative fixing and without version bump?

Sorry for the using unsafe function, my mistake.

byorgey commented 8 years ago

Sure, you can close this pull request and make a new one if you want. No worries!

On 11:25am, Tue, Jul 21, 2015 Alexander Krupenkin notifications@github.com wrote:

Thank you! drop 1 is really simple! Could I create new PR with alternative fixing and without version bump?

Sorry for the using unsafe function, my mistake.

— Reply to this email directly or view it on GitHub https://github.com/byorgey/haxr/pull/9#issuecomment-123392469.