From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751430AbaLXGnG (ORCPT ); Wed, 24 Dec 2014 01:43:06 -0500 Received: from mga11.intel.com ([192.55.52.93]:21596 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750740AbaLXGnE (ORCPT ); Wed, 24 Dec 2014 01:43:04 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.07,637,1413270000"; d="scan'208";a="652546265" Date: Wed, 24 Dec 2014 14:41:39 +0800 From: kbuild test robot To: Pravin B Shelar Cc: kbuild-all@01.org, dev@openvswitch.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] openvswitch: fix odd_ptr_err.cocci warnings Message-ID: <20141224064138.GA11360@athens> References: <201412241437.BpaCtsIE%fengguang.wu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201412241437.BpaCtsIE%fengguang.wu@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: fengguang.wu@intel.com X-SA-Exim-Scanned: No (on bee); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org net/openvswitch/vport-gre.c:188:5-11: inconsistent IS_ERR and PTR_ERR, PTR_ERR on line 189 PTR_ERR should access the value just tested by IS_ERR Semantic patch information: There can be false positives in the patch case, where it is the call IS_ERR that is wrong. Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci CC: Pravin B Shelar Signed-off-by: Fengguang Wu --- vport-gre.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/net/openvswitch/vport-gre.c +++ b/net/openvswitch/vport-gre.c @@ -186,7 +186,7 @@ static int gre_tnl_send(struct vport *vp /* Push Tunnel header. */ skb = __build_header(skb, tunnel_hlen); if (IS_ERR(skb)) { - err = PTR_ERR(rt); + err = PTR_ERR(skb); skb = NULL; goto err_free_rt; } From mboxrd@z Thu Jan 1 00:00:00 1970 From: kbuild test robot Subject: [PATCH] openvswitch: fix odd_ptr_err.cocci warnings Date: Wed, 24 Dec 2014 14:41:39 +0800 Message-ID: <20141224064138.GA11360@athens> References: <201412241437.BpaCtsIE%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: dev-yBygre7rU0TnMu66kgdUjQ@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kbuild-all-JC7UmRfGjtg@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Pravin B Shelar Return-path: Content-Disposition: inline In-Reply-To: <201412241437.BpaCtsIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-yBygre7rU0TnMu66kgdUjQ@public.gmane.org Sender: "dev" List-Id: netdev.vger.kernel.org bmV0L29wZW52c3dpdGNoL3Zwb3J0LWdyZS5jOjE4ODo1LTExOiBpbmNvbnNpc3RlbnQgSVNfRVJS IGFuZCBQVFJfRVJSLCBQVFJfRVJSIG9uIGxpbmUgMTg5CgogUFRSX0VSUiBzaG91bGQgYWNjZXNz IHRoZSB2YWx1ZSBqdXN0IHRlc3RlZCBieSBJU19FUlIKClNlbWFudGljIHBhdGNoIGluZm9ybWF0 aW9uOgogVGhlcmUgY2FuIGJlIGZhbHNlIHBvc2l0aXZlcyBpbiB0aGUgcGF0Y2ggY2FzZSwgd2hl cmUgaXQgaXMgdGhlIGNhbGwKIElTX0VSUiB0aGF0IGlzIHdyb25nLgoKR2VuZXJhdGVkIGJ5OiBz Y3JpcHRzL2NvY2NpbmVsbGUvdGVzdHMvb2RkX3B0cl9lcnIuY29jY2kKCkNDOiBQcmF2aW4gQiBT aGVsYXIgPHBzaGVsYXJAbmljaXJhLmNvbT4KU2lnbmVkLW9mZi1ieTogRmVuZ2d1YW5nIFd1IDxm ZW5nZ3Vhbmcud3VAaW50ZWwuY29tPgotLS0KCiB2cG9ydC1ncmUuYyB8ICAgIDIgKy0KIDEgZmls ZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQoKLS0tIGEvbmV0L29wZW52 c3dpdGNoL3Zwb3J0LWdyZS5jCisrKyBiL25ldC9vcGVudnN3aXRjaC92cG9ydC1ncmUuYwpAQCAt MTg2LDcgKzE4Niw3IEBAIHN0YXRpYyBpbnQgZ3JlX3RubF9zZW5kKHN0cnVjdCB2cG9ydCAqdnAK IAkvKiBQdXNoIFR1bm5lbCBoZWFkZXIuICovCiAJc2tiID0gX19idWlsZF9oZWFkZXIoc2tiLCB0 dW5uZWxfaGxlbik7CiAJaWYgKElTX0VSUihza2IpKSB7Ci0JCWVyciA9IFBUUl9FUlIocnQpOwor CQllcnIgPSBQVFJfRVJSKHNrYik7CiAJCXNrYiA9IE5VTEw7CiAJCWdvdG8gZXJyX2ZyZWVfcnQ7 CiAJfQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkZXYg bWFpbGluZyBsaXN0CmRldkBvcGVudnN3aXRjaC5vcmcKaHR0cDovL29wZW52c3dpdGNoLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2Rldgo=