montrehack / montrehack.github.com

The MontréHack website
https://montrehack.ca
21 stars 13 forks source link

malware reversing (2016/09) writeup #20

Closed pmbureau closed 7 years ago

obilodeau commented 7 years ago

Looks good from a diff perspective. Let's merge and see if everything is all right in a browser.

obilodeau commented 7 years ago

I see two minor glitches from a quick review:

Nothing major but it's too late for me to look at it right now.

pmbureau commented 7 years ago

Awesome, thanks for spotting this. I have fixed these two issues and pushed them.

On Tue, Dec 13, 2016 at 9:42 PM Olivier Bilodeau notifications@github.com wrote:

I see two minor glitches from a quick review:

  • binary download link broken
  • header of the index.md file seems to be displayed in file

Nothing major but it's too late for me to look at it right now.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/montrehack/montrehack.github.com/pull/20#issuecomment-266947953, or mute the thread https://github.com/notifications/unsubscribe-auth/ABGb1OfJORZQ-O7KCnKgy1pUKszbbnzUks5rH4G6gaJpZM4LLUPK .