Add more comments

Author: qhdwight

buffer_overflow/Makefile

@@ -1,6 +1,10 @@
 CC=gcc
 CFLAGS=-I.
 
+# Turns out modern compilers have a lot of default settings to prevent vulnerable code
+# -fno-stack-protector: Disable stack canary: https://ctf101.org/binary-exploitation/stack-canaries/
+# -no-pie: Disables ASLR essentially, loads executable to the same address every time
+# -Wno-stringop-overflow: Disable complaining about fgets overflow
 buffer_overflow: buffer_overflow.c
 	$(CC) -o buffer_overflow buffer_overflow.c -fno-stack-protector -no-pie -Wno-stringop-overflow
 

buffer_overflow/buffer_overflow.c

@@ -1,5 +1,5 @@
-#include <stdio.h> // for puts and gets
-#include <unistd.h>
+#include <stdio.h> // For puts and fgets
+#include <unistd.h> // For execve
 
 void access_vault() {
   puts("Access granted");