From 94d923de400fc1c6a50b67a55ce700251f93ab05 Mon Sep 17 00:00:00 2001 From: Oleg Drokin Date: Tue, 12 Aug 2014 08:34:23 -0400 Subject: [PATCH] LU-4423 obdecho: bitwise vs logical typo Bitwise AND was intended here obviously. From upstream kernel commit 96d4b0d5636b1810042eabedf766f35b548f1672 Change-Id: I5f05d14843910c2288c14c55e62615f030a7f445 Signed-off-by: Dan Carpenter Signed-off-by: Oleg Drokin Reviewed-on: http://review.whamcloud.com/11410 Reviewed-by: Dmitry Eremin Reviewed-by: James Simmons Tested-by: Jenkins Tested-by: Maloo --- lustre/obdecho/echo.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lustre/obdecho/echo.c b/lustre/obdecho/echo.c index 5ece81d..122eb70 100644 --- a/lustre/obdecho/echo.c +++ b/lustre/obdecho/echo.c @@ -127,10 +127,10 @@ static int echo_create(const struct lu_env *env, struct obd_export *exp, return -EINVAL; } - if (!(oa->o_mode && S_IFMT)) { - CERROR("echo obd: no type!\n"); - return -ENOENT; - } + if (!(oa->o_mode & S_IFMT)) { + CERROR("echo obd: no type!\n"); + return -ENOENT; + } if (!(oa->o_valid & OBD_MD_FLTYPE)) { CERROR("invalid o_valid "LPX64"\n", oa->o_valid); -- 1.8.3.1