[autobackport: sssd-2-9] IPA: memory leak fixed

[sssd-bot]

This is an automatic backport of PR#8632 IPA: memory leak fixed to branch sssd-2-9, created by @alexey-tikhonov.

Please make sure this backport is correct.

Note

The commits were cherry-picked without conflicts.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr8632-to-sssd-2-9
git checkout SSSD-sssd-backport-pr8632-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr8632-to-sssd-2-9 --force

---

Original commits
958a186 - IPA: memory leak fixed

Backported commits

• 05ed94a - IPA: memory leak fixed

---

Original Pull Request Body

Result allocated by sss_base64_decode() on NULL context was never freed.

[gemini-code-assist]

Code Review

This pull request modifies src/providers/ipa/ipa_s2n_exop.c to use a specific memory context for base64 decoding and introduces explicit memory cleanup in error handling paths. The review feedback correctly identifies that the newly added deallocation calls for name and values are redundant and would result in double-free errors, as these variables are already managed at the end of the processing loop.

[sumit-bose]

Hi,

backport and original version match, ACK.

I think Gemini's comments are wrong because it didn't take the return immediately after the free-s into account.

bye,
Sumit

[sssd-bot]

The pull request was accepted by @alexey-tikhonov with the following PR CI status:

---

🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-9) (success)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)

---

There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.