Segmentation fault if for-loop is removed from jwt.c

CyberNinjas / pam_aad

Azure Active Directory PAM Module

GNU General Public License v3.0

54 stars 19 forks source link

Segmentation fault if for-loop is removed from jwt.c #1

Closed SJCaldwell closed 7 years ago

SJCaldwell commented 7 years ago

Lines: 61-62

    for(i = 0; i < num_tokens; i++){
    }

The following code appears to do nothing, but if I get rid of it there is a segfault.

It's not hurting development progress anyway, but something has to be done about it.

SJCaldwell commented 7 years ago

Turns out this only applies to the main function of jwt.c, which isn't relevant to for the pam shared library since main isn't included in it. I'll keep looking into it, but it's not a bug in the project.