From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753218AbZBPPGp (ORCPT ); Mon, 16 Feb 2009 10:06:45 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754835AbZBPPFP (ORCPT ); Mon, 16 Feb 2009 10:05:15 -0500 Received: from cmpxchg.org ([85.214.51.133]:49285 "EHLO cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752370AbZBPPFN (ORCPT ); Mon, 16 Feb 2009 10:05:13 -0500 Message-Id: <20090216144725.976425091@cmpxchg.org> User-Agent: quilt/0.47-1 Date: Mon, 16 Feb 2009 15:29:32 +0100 From: Johannes Weiner To: Andrew Morton Cc: Pekka Enberg , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Steve French Subject: [patch 6/8] cifs: use kzfree() References: <20090216142926.440561506@cmpxchg.org> Content-Disposition: inline; filename=cifs-use-kzfree.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Use kzfree() instead of memset() + kfree(). Signed-off-by: Johannes Weiner Cc: Steve French --- fs/cifs/connect.c | 7 ++----- fs/cifs/misc.c | 12 ++++-------- 2 files changed, 6 insertions(+), 13 deletions(-) --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -2433,11 +2433,8 @@ mount_fail_check: out: /* zero out password before freeing */ if (volume_info) { - if (volume_info->password != NULL) { - memset(volume_info->password, 0, - strlen(volume_info->password)); - kfree(volume_info->password); - } + if (volume_info->password != NULL) + kzfree(volume_info->password); kfree(volume_info->UNC); kfree(volume_info->prepath); kfree(volume_info); --- a/fs/cifs/misc.c +++ b/fs/cifs/misc.c @@ -97,10 +97,8 @@ sesInfoFree(struct cifsSesInfo *buf_to_f kfree(buf_to_free->serverOS); kfree(buf_to_free->serverDomain); kfree(buf_to_free->serverNOS); - if (buf_to_free->password) { - memset(buf_to_free->password, 0, strlen(buf_to_free->password)); - kfree(buf_to_free->password); - } + if (buf_to_free->password) + kzfree(buf_to_free->password); kfree(buf_to_free->domainName); kfree(buf_to_free); } @@ -132,10 +130,8 @@ tconInfoFree(struct cifsTconInfo *buf_to } atomic_dec(&tconInfoAllocCount); kfree(buf_to_free->nativeFileSystem); - if (buf_to_free->password) { - memset(buf_to_free->password, 0, strlen(buf_to_free->password)); - kfree(buf_to_free->password); - } + if (buf_to_free->password) + kzfree(buf_to_free->password); kfree(buf_to_free); }