From patchwork Tue May 8 20:39:37 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Vivier X-Patchwork-Id: 910452 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=nongnu.org (client-ip=2001:4830:134:3::11; helo=lists.gnu.org; envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=vivier.eu Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 40gWZs6ckrz9s37 for ; Wed, 9 May 2018 06:40:19 +1000 (AEST) Received: from localhost ([::1]:53250 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fG9PD-0006TL-Ds for incoming@patchwork.ozlabs.org; Tue, 08 May 2018 16:40:15 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53020) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fG9Op-0006T2-Ei for qemu-devel@nongnu.org; Tue, 08 May 2018 16:39:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fG9Om-0007LB-At for qemu-devel@nongnu.org; Tue, 08 May 2018 16:39:51 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:50661) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fG9Om-0007L5-18 for qemu-devel@nongnu.org; Tue, 08 May 2018 16:39:48 -0400 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue104 [212.227.15.183]) with ESMTPSA (Nemesis) id 0MF4EF-1f4ura15PX-00GK26; Tue, 08 May 2018 22:39:45 +0200 From: Laurent Vivier To: qemu-devel@nongnu.org Date: Tue, 8 May 2018 22:39:37 +0200 Message-Id: <20180508203937.16796-1-laurent@vivier.eu> X-Mailer: git-send-email 2.14.3 X-Provags-ID: V03:K1:Mvd0RE6A/nNAQqIG9NsU6XbgUeiHWNZ6CZWDzGi8reqHbA2Iq/p I+CEvfJ3TJYlfNNazul1M1+dbRC2xPYTiPotuLDdnGLg9QKXB2GV63fbPOxxKl+fKBugXzO P6ZiQEwXR332ovyIijj/Y9yyDF4xr7mJwfSC2ZxZl+2ngsdu/N/n7bq4p0D/Gyl+dO6Sta1 6Gp6iwvaxHh7/UjY6KJRQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:DaOm55S8w4A=:OQcfN3a5c/n8k/aW276uDK EdrBmli0CVZd3/m6uohoqPVI3ZeTJAbcAhQZoJ/LndWd0fIThso27BNKswolyG5T6WiozuOxE VBbjokpTJxYe6NN2jNboEBHBttpeOSsvkLc8T6u2m3hC5YhZEmuT1/l7xW7DXe+PfwhWfInp+ tJKM2ZHrRU1ylq+t/6vv9jpw+TyoI+cNHoms9BT1pa/PA9DPK0Zb8rvZOMjRbIQ8F/8z6r5ZX YR14Q5sxrpccBqO7pgSrwWigKMbNqMxxvdWaUFwBJp7jmWnIWoA3DNLnIwrrK9SoUhO1qZK18 WGIUyRliCvdVtrOgKQrqK54LjJlW7YPCxGhZFVAkZ2ODhkwJtz+9UmHbDgDfq4OKTe2yxAO9T u45t4VIujYXaNn4aCQTOfSe5Q3RACG+FA4nyWEXuT1gupUDJUTm4in1ZMblIu7GmcZl5DG73Y msE7i8J4ys+xlmH3k36AufJ4C9b0tYEsXoz8k0gMOspDPvmd9UzJ8RVecXjFHaZgJseDlNLjc 8rC6dh92MiVjrk1mhQIjGuvk/I6BMaLP8iV7iPHzrsnuw/nZrr8Hsf9zoD4fMZrG7Hd/3b1Rt WjlZz+saQWdlyae1ofgiCWlapO03MFn+FGMRuAaugxyWCufWvDSvfQrxs6w1hKb07Jn2HpeYP sk2FaLxjseuOtxrNZeofyDzQssfiEyObpKEDIiLi6SzD6ryB+xpbkS6/1rFSTE1zr3bw= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 217.72.192.73 Subject: [Qemu-devel] [PATCH] m68k: fix floatx80_mod() (Coverity CID1390568) X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Laurent Vivier Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" Update the variable checked by the loop condition (expDiff). Backport the update from Previous. Fixes: 591596b77a ("target/m68k: add fmod/frem") Signed-off-by: Laurent Vivier Reviewed-by: Thomas Huth --- target/m68k/softfloat.c | 1 + 1 file changed, 1 insertion(+) diff --git a/target/m68k/softfloat.c b/target/m68k/softfloat.c index e41b07d042..d093997219 100644 --- a/target/m68k/softfloat.c +++ b/target/m68k/softfloat.c @@ -103,6 +103,7 @@ floatx80 floatx80_mod(floatx80 a, floatx80 b, float_status *status) mul64To128(bSig, qTemp, &term0, &term1); sub128(aSig0, aSig1, term0, term1, &aSig0, &aSig1); shortShift128Left(aSig0, aSig1, 62, &aSig0, &aSig1); + expDiff -= 62; } expDiff += 64; if (0 < expDiff) {