Menu

#12 large HTML comments are removed and replaced ba a random character

v1.0 (example)
closed-fixed
nobody
None
5
2015-10-16
2015-10-10
No

See subject. :-)

1 Attachments

Discussion

  • Jochen Hoenicke

    Jochen Hoenicke - 2015-10-10

    Hi, I saw the bug report you sent to me earlier, but I only had time to investigate this now.

    The problem is caused by Microsoft's non-standard (or buggy) implementation of snprintf. I recompiled the windows version to use the snprintf that comes with the htp sources. A warning let me suspect that htp's snprintf implementation is buggy for 64-bit windows (where long is smaller than the size of a pointer), so this time there is only a 32-bit binary. It should also run on 64bit windows.

    I uploaded the new binary. I didn't change the version number. The code is the same only the compile options changed. For the record, the options are:

    CC = i686-w64-mingw32-gcc
    LINK = $(CC)
    STRIP = i686-w64-mingw32-strip
    OBJEXT = o
    EXEEXT = .exe
    OS = Win32
    DEFINES = -DHAVE_VA_COPY -D_GNU_SOURCE=1
    
     

    Last edit: Jochen Hoenicke 2015-10-16
    • Robert Böck

      Robert Böck - 2015-10-16

      Sorry, I can't confirm that the problem is fixed now. The 1.18 Win32 binary didn't have this problem.

       
  • Jochen Hoenicke

    Jochen Hoenicke - 2015-10-10
    • status: open --> closed-fixed
     
  • Jochen Hoenicke

    Jochen Hoenicke - 2015-10-16

    Strange, although I'm sure I tested it, I still had the same problem when I tested today. I'm not sure what happened, maybe a working binary at some place that took precedence. I compiled again this
    time with

    DEFINES=-DHAVE_SNPRINTF -DPREFER_PORTABLE_SNPRINTF -DHAVE_VA_COPY -D_GNU_SOURCE=1
    

    Can you test if it works now?

     
    • Robert Böck

      Robert Böck - 2015-10-20

      It works now. Thank you. :-)

       

Log in to post a comment.