#9936 Code analysis issue in ipa-4.9
Closed: fixed by frenaud. Opened by rjeffman.

  1. Defect type: UNINIT

  2. freeipa-4.9.13/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c:132:5: var_decl: Declaring variable "sdn" without initializer.

  3. freeipa-4.9.13/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c:337:5: uninit_use: Using uninitialized value "sdn".
#   335|       if (krbLastPwdChange) slapi_ch_free_string(&krbLastPwdChange);
#   336|       if (fqdn) slapi_ch_free_string(&fqdn);
#   337|->     if (sdn) slapi_sdn_free(&sdn);
#   338|   
#   339|       LOG("%s", errMesg ? errMesg : "success\n");

This is an easy fix.

Same issue exists on other branches (master / ipa-4-13 / ipa-4-12)

Metadata Update from @frenaud:
- Issue assigned to frenaud

Metadata Update from @frenaud:
- Custom field on_review adjusted to https://github.com/freeipa/freeipa/pull/8155

master:

  • 549d64ab30cab2ca5907e43680407c58f4baa5b8 ipa-join: initialize pointer

ipa-4-9:

  • b7d9e0e5c325282eb7f7c7d51fbd8708814479ce ipa-join: initialize pointer

ipa-4-12:

  • 23a107df1b77db415cb9f6a5ccad81d8062d6c3f ipa-join: initialize pointer

ipa-4-13:

  • 7cc96e42683a6d3ec9f2dc2a19e99330b6f3ce58 ipa-join: initialize pointer

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

Metadata