1.malloc forget to free memory 2.finishing code