#9365 Covscan issues: usage of free() instead of krb5_free_enctypes()
Closed: fixed 24 days ago by sumenon. Opened 2 years ago by rjeffman.

The following issues were raised by Covscan as ipa_krb5_free_ktypes() frees ktypes with free() rather than krb5_free_enctypes().

 2. freeipa-4.9.11/util/ipa_krb5.c:890: alloc: Allocation of memory which must be freed using "krb5_free_enctypes".
15. freeipa-4.9.11/util/ipa_krb5.c:909: free: Calling "ipa_krb5_free_ktypes" frees "ktypes" using "free" but it should have been freed using "krb5_free_enctypes".
#   907|           }
#   908|   
#   909|->         ipa_krb5_free_ktypes(krbctx, ktypes);
#   910|   
#   911|           nkeys = i;
 9. freeipa-4.9.11/client/ipa-getkeytab.c:1012: alloc: Allocation of memory which must be freed using "krb5_free_enctypes".
18. freeipa-4.9.11/client/ipa-getkeytab.c:1027: free: Calling "ipa_krb5_free_ktypes" frees "ktypes" using "free" but it should have been freed using "krb5_free_enctypes".
#  1025|            fprintf(stdout, "%s\n", enc);
#  1026|        }
#  1027|->      ipa_krb5_free_ktypes(krbctx, ktypes);
#  1028|        exit (0);
#  1029|    }

Metadata Update from @rcritten:
- Issue assigned to rcritten

a month ago

master:

  • 111e0f0 Fix some issues identified by a static analyzer

ipa-4-12:

  • 777f4c0 Fix some issues identified by a static analyzer

Metadata Update from @sumenon:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

24 days ago

Log in to comment on this ticket.

Metadata