Fix double free with lval_del

orangeduck / BuildYourOwnLisp

Learn C and build your own programming language in under 1000 lines of code!

http://www.buildyourownlisp.com/

Other

2.9k stars 394 forks source link

Fix double free with lval_del #33

Closed radiofreejohn closed 10 years ago

radiofreejohn commented 10 years ago

Reported on #31 lval_del(a) is called twice when dividing by zero. Removing the lval_del on line 459 since it's called on 471.

Sorry for not putting both these pulls in the same request.

orangeduck commented 10 years ago

Hi,

Good catch. Would it be possible to propagate this change to the following chapter's code as well as the source code embedded in the HTML web pages for all relevant chapters. Sorry I know it is a real pain!

Thanks,

Dan

radiofreejohn commented 10 years ago

Sure thing