Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760311AbcLPKqd (ORCPT ); Fri, 16 Dec 2016 05:46:33 -0500 Received: from aserp1040.oracle.com ([141.146.126.69]:17461 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757063AbcLPKqO (ORCPT ); Fri, 16 Dec 2016 05:46:14 -0500 Date: Fri, 16 Dec 2016 13:45:24 +0300 From: Dan Carpenter To: Mike Marshall Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [patch] orangefs: cleanup orangefs_debugfs_new_client_string() Message-ID: <20161216104524.GA2637@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1946 Lines: 65 Several small things in this function: 1) If copy to user fails we should return -EFAULT not -EIO 2) Don't print an error message, just fail. It's annoying to let the users fill up dmesg and especially for something small like this. 3) Remove a stray tab. 4) Preserve the error code if orangefs_prepare_debugfs_help_string() fails. 5) "return 0;" is more explicit and clear than "return ret;". Signed-off-by: Dan Carpenter diff --git a/fs/orangefs/orangefs-debugfs.c b/fs/orangefs/orangefs-debugfs.c index 27e75cf28b3a..409fa6b0d339 100644 --- a/fs/orangefs/orangefs-debugfs.c +++ b/fs/orangefs/orangefs-debugfs.c @@ -966,15 +966,9 @@ int orangefs_debugfs_new_client_string(void __user *arg) { int ret; - ret = copy_from_user(&client_debug_array_string, - (void __user *)arg, - ORANGEFS_MAX_DEBUG_STRING_LEN); - - if (ret != 0) { - pr_info("%s: CLIENT_STRING: copy_from_user failed\n", - __func__); - return -EIO; - } + if (copy_from_user(&client_debug_array_string, arg, + ORANGEFS_MAX_DEBUG_STRING_LEN)) + return -EFAULT; /* * The real client-core makes an effort to ensure @@ -988,17 +982,18 @@ int orangefs_debugfs_new_client_string(void __user *arg) */ client_debug_array_string[ORANGEFS_MAX_DEBUG_STRING_LEN - 1] = '\0'; - + pr_info("%s: client debug array string has been received.\n", __func__); if (!help_string_initialized) { /* Build a proper debug help string. */ - if (orangefs_prepare_debugfs_help_string(0)) { + ret = orangefs_prepare_debugfs_help_string(0); + if (ret) { gossip_err("%s: no debug help string \n", __func__); - return -EIO; + return ret; } } @@ -1011,7 +1006,7 @@ int orangefs_debugfs_new_client_string(void __user *arg) help_string_initialized++; - return ret; + return 0; } int orangefs_debugfs_new_debug(void __user *arg)