Messages in this thread Patch in this message | | | From | Thierry Reding <> | Subject | [PATCH] fs/cifs: Fix build warnings | Date | Mon, 14 Jul 2014 12:38:49 +0200 |
| |
From: Thierry Reding <treding@nvidia.com>
The sess_alloc_buffer(), sess_free_buffer(), sess_establish_session() and sess_sendreceive() functions are only used by sess_auth_lanman(), therefore should be guarded by the same #ifdef to prevent the following compiler warnings:
fs/cifs/sess.c:542:1: warning: 'sess_alloc_buffer' defined but not used [-Wunused-function] sess_alloc_buffer(struct sess_data *sess_data, int wct) ^ fs/cifs/sess.c:580:1: warning: 'sess_free_buffer' defined but not used [-Wunused-function] sess_free_buffer(struct sess_data *sess_data) ^ fs/cifs/sess.c:589:1: warning: 'sess_establish_session' defined but not used [-Wunused-function] sess_establish_session(struct sess_data *sess_data) ^ fs/cifs/sess.c:621:1: warning: 'sess_sendreceive' defined but not used [-Wunused-function] sess_sendreceive(struct sess_data *sess_data) ^
Signed-off-by: Thierry Reding <treding@nvidia.com> --- fs/cifs/sess.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c index db6ecdf0974c..5fb7606b81ef 100644 --- a/fs/cifs/sess.c +++ b/fs/cifs/sess.c @@ -538,6 +538,7 @@ struct sess_data { struct kvec iov[3]; }; +#ifdef CONFIG_CIFS_WEAK_PW_HASH static int sess_alloc_buffer(struct sess_data *sess_data, int wct) { @@ -644,7 +645,6 @@ sess_sendreceive(struct sess_data *sess_data) * in proc/fs/cifs or via mount parm. Unfortunately this is * needed for old Win (e.g. Win95), some obscure NAS and OS/2 */ -#ifdef CONFIG_CIFS_WEAK_PW_HASH static void sess_auth_lanman(struct sess_data *sess_data) { -- 2.0.1
| |