1. Wez Furlong
  2. atomicparsley
  3. Issues
Issue #37 resolved

AtomicParsley file -t -T segfaults

Anonymous created an issue


Comments (3)

  1. Oleg Oshmyan

    -t calls APar_FreeMemory in the option parsing loop (but doesn’t immediately exit) in real_main. Simply removing this call fixes the issue and doesn’t seem to break anything else (neither empirically nor from a quick look through the source code).

    Inverting the option order to -T -t reveals another bug: -T doesn’t close the file after opening it.

    I want to note that -T defaults to implying --DeepScan if it’s the first option, but in -t -T it isn’t, so deep scanning remains off unless you explicitly enable it with --DeepScan -t -T.

    I’ll make a pull request in a moment.

  2. Log in to comment