diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-02-26 18:19:59 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-03-02 15:29:17 +0100 |
| commit | cac22be9e58abdcf6c3bf66190fba0f7cb6f490e (patch) | |
| tree | af824a62dd1dab9f11e33e5c5994afd801fb8513 /src/sss_client | |
| parent | c820e6db26426c9f74a0e8f8ff9d9cf79d57406a (diff) | |
| download | sssd-cac22be9e58abdcf6c3bf66190fba0f7cb6f490e.tar.gz sssd-cac22be9e58abdcf6c3bf66190fba0f7cb6f490e.tar.xz sssd-cac22be9e58abdcf6c3bf66190fba0f7cb6f490e.zip | |
Remove useless assignment to function parameter
Reported by: cppcheck
void free_fun(struct info *info)
free(info->name);
free(info);
info = NULL;
^^^^^^^^^^^
Assignment to function parameter has no effect outside the function.
Reviewed-by: Pavel Reichl <preichl@redhat.com>
Diffstat (limited to 'src/sss_client')
| -rw-r--r-- | src/sss_client/pam_sss.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/sss_client/pam_sss.c b/src/sss_client/pam_sss.c index 28a36d5af..6105cac0e 100644 --- a/src/sss_client/pam_sss.c +++ b/src/sss_client/pam_sss.c @@ -124,7 +124,6 @@ static void logger(pam_handle_t *pamh, int level, const char *fmt, ...) { static void free_exp_data(pam_handle_t *pamh, void *ptr, int err) { free(ptr); - ptr = NULL; } static void close_fd(pam_handle_t *pamh, void *ptr, int err) |