Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

KRB5: mistype fix #7752

Closed
wants to merge 1 commit into from
Closed

KRB5: mistype fix #7752

wants to merge 1 commit into from

Conversation

alexey-tikhonov
Copy link
Member

Fixes:

 *** CID 515655:  Uninitialized variables  (UNINIT)
 /home/runner/work/sssd/sssd/src/providers/krb5/krb5_child.c: 2435 in
 get_and_save_tgt()
 2429             goto done;
 2430         }
 2431
 2432         /* Make sure ccache is created and written as the user */
 2433         kerr = switch_to_user();
 2434         if (kerr != EOK) {
  >>>     CID 515655:  Uninitialized variables  (UNINIT)
  >>>     Using uninitialized value "ret" when calling "sss_debug_fn".
 2435             DEBUG(SSSDBG_CRIT_FAILURE, "Failed to switch to user
 IDs: %d\n", ret);
 2436             goto done;
 2437         }
 2438
 2439         log_process_caps("Saving ccache");
 2440

@alexey-tikhonov
KRB5: mistype fix
82cc642 
Fixes:
```
 *** CID 515655:  Uninitialized variables  (UNINIT)
 /home/runner/work/sssd/sssd/src/providers/krb5/krb5_child.c: 2435 in
 get_and_save_tgt()
 2429             goto done;
 2430         }
 2431
 2432         /* Make sure ccache is created and written as the user */
 2433         kerr = switch_to_user();
 2434         if (kerr != EOK) {
  >>>     CID 515655:  Uninitialized variables  (UNINIT)
  >>>     Using uninitialized value "ret" when calling "sss_debug_fn".
 2435             DEBUG(SSSDBG_CRIT_FAILURE, "Failed to switch to user
 IDs: %d\n", ret);
 2436             goto done;
 2437         }
 2438
 2439         log_process_caps("Saving ccache");
 2440
```
@jakub-vavra-cz jakub-vavra-cz self-requested a review December 6, 2024 13:29
@jakub-vavra-cz jakub-vavra-cz self-assigned this Dec 6, 2024
jakub-vavra-cz
jakub-vavra-cz approved these changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@jakub-vavra-cz jakub-vavra-cz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks alright.

@alexey-tikhonov alexey-tikhonov added the Ready to push Ready to push label Dec 6, 2024
@alexey-tikhonov
Copy link
Member Author

Pushed PR: #7752

  • master
  • sssd-2-10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakub-vavra-cz jakub-vavra-cz jakub-vavra-cz approved these changes

Assignees

@jakub-vavra-cz jakub-vavra-cz

Labels
branch: sssd-2-10 Pushed Trivial
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexey-tikhonov @jakub-vavra-cz