RPISEC / MBE

Course materials for Modern Binary Exploitation by RPISEC
BSD 2-Clause "Simplified" License
5.43k stars 881 forks source link

Update CMakeLists.txt #43

Open blacksheeep opened 3 years ago

blacksheeep commented 3 years ago

The flag to disable the Canary is named wrong when adding to NO_MITS_FLAGS in CMakeLists.txt. It is defined as DISABLE_CANARY_FLAG but added as DISABLE_CANARIES_FLAG.

This leads to all binaries build with NO_MITS_FLAGS still having a Canary.

This pull requests fixes the issue by changing the flag name consistendly to DISABLE_CANARY_FLAG, in the definition as well as when it is added to NO_MITS_FLAGS.