From 65710dae847273edb99495241e6d7b7acd303c5f Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Sat, 6 Feb 2010 12:56:19 +0100 Subject: pam_winbind: Remove a nested if-statement. Bo, please check! --- nsswitch/pam_winbind.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/nsswitch/pam_winbind.c b/nsswitch/pam_winbind.c index fcad15c0003..82c8c4d0e25 100644 --- a/nsswitch/pam_winbind.c +++ b/nsswitch/pam_winbind.c @@ -1199,10 +1199,8 @@ static bool winbind_name_list_to_sid_string_list(struct pwb_context *ctx, * It is malformated parameter here, overwrite the last ','. */ len = strlen(sid_list_buffer); - if (len) { - if (sid_list_buffer[len - 1] == ',') { - sid_list_buffer[len - 1] = '\0'; - } + if ((len != 0) && (sid_list_buffer[len - 1] == ',')) { + sid_list_buffer[len - 1] = '\0'; } } -- cgit