search

google / google-authenticator-libpam

Apache License 2.0
1.76k stars 281 forks source link

Memory leak #150

Closed QiAnXinCodeSafe closed 4 years ago

QiAnXinCodeSafe commented 4 years ago

Memory is allocated https://github.com/google/google-authenticator-libpam/blob/fe17bd27235a09e3eb7d755afac8c33fab617ac9/src/base32_prog.c#L130

never free it https://github.com/google/google-authenticator-libpam/blob/fe17bd27235a09e3eb7d755afac8c33fab617ac9/src/base32_prog.c#L158

ThomasHabets commented 4 years ago

Again, just like #149: No, that's not how memory allocation works. Please fix whatever broken code analyzer you're using.