olehlong / jwtd

D implementation of JSON Web Token.
MIT License
30 stars 13 forks source link

Fix a few resource leaks in error conditions #17

Closed oaLightning closed 7 years ago

oaLightning commented 7 years ago

These are all fixes Ali Çehreli made while working with the library.

The main fix is 795d387 which fixes a few potential resource leaks in error conditions and makes the code more "D" in some parts. Specifically this would happen in jwt_openssl in cases where an exception was thrown (and the assumption that the calling function would release the memory for the eckey variable). 2f86a2f is a small fix to the README.md so that bullet points will be the same size. In b788e46 Ali moved a few unit tests around and added some more to increase the test coverage.