From patchwork Wed May 29 22:13:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Tomas_H=C3=A4rdin?= X-Patchwork-Id: 49370 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a59:8f0d:0:b0:460:55fa:d5ed with SMTP id i13csp952464vqu; Wed, 29 May 2024 15:14:01 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCWjgkXiRQIHEx3mDWvrkL8UqXGHjTpL7IB3yevJx20E47ID1czVejzQMwWjWHBNHfR9h8U76iBmU9FskbUwuagMgXW6UeIhQ2CuUw== X-Google-Smtp-Source: AGHT+IE6cP9oGZFvhqKeRVES/tCWE0NqUJty1PF2Kz1oxJNDZL412SqqrpHuoodttcLzC7aQQBAW X-Received: by 2002:a17:907:9717:b0:a5a:7a1:5da6 with SMTP id a640c23a62f3a-a65e90f780amr23642966b.3.1717020841232; Wed, 29 May 2024 15:14:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1717020841; cv=none; d=google.com; s=arc-20160816; b=p8HAhsHmAkCgQ1RUAFzySg22qBpWQTB5btg+evqrGBMKkA8kun9hAM5le55rzuvm0u 8NLvz3gw3zRO6C74ns38FBUi+dLDOpWIaH7kz1eXSXDLyIPG6EIGTC7o0VBfX91pqLHQ MB9/thO+489ISUZcHJ4yPE75AH6v6ItKu9nvhWDJIXuWeF/MDychgisL1zTpiL/IqCcy AeYA/J7UIPnMYWpmwwSlj9sxEnBHKTrO8uuFgY3HYbg4rwP3skeGQhyv960Y2KbPapnc xkerus1wXbIRvjFid8sHOCyvc/y4nMPsHMQksI3EnCgceV6OdiNREodVYByM1pj/Kffw 8yOg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:reply-to:list-subscribe:list-help:list-post :list-archive:list-unsubscribe:list-id:precedence:subject :mime-version:user-agent:references:in-reply-to:date:to:from :message-id:delivered-to; bh=aeHmulfQCvdya+ENumELAnWYXkgXOMNRsMBcbzCEn8k=; fh=e5zN9xSzcxLA6bGo3lF+CqTbY/oLwzApV03EO/RBfgQ=; b=JdycGn00hHjOGvxIspHWzl4aTK6eYxGRIN3UpB+WEmAGWjLX0OcP7Lzxx1T5eCrBur RxHf/UTeJisznK4NBYTm4imhrz1yET7cpy4olppT7/BF0UH4MvYWkxYjMKFhY/OpXuGJ CwpbB/AAOpF5OgGshsmalTmHoPsLeKBP0cE4eq1ShIEIYfMqvf3XecwAVKXvzJQ0Z34s +LKotF0u+1yn/ofUDv3/47JYmDJfdnnIBAWqEDAJwaScccGqxQfiar2+SjX4PUt7+B+e 73HPL1Y41E/kYJT3oi+dyaLfDBSu5iHREZ+J5ttjEEpJM8QAoN/G9dORrSCbMoMR4zxV PlJQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id a640c23a62f3a-a626cc642dcsi648912066b.504.2024.05.29.15.14.00; Wed, 29 May 2024 15:14:01 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C935868D406; Thu, 30 May 2024 01:13:58 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from glom.nmugroup.com (glom.nmugroup.com [193.183.80.6]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 565D468BFFF for ; Thu, 30 May 2024 01:13:51 +0300 (EEST) Received: from localhost (localhost [127.0.0.1]) by glom.nmugroup.com (Postfix) with ESMTP id 0346754292DE for ; Thu, 30 May 2024 00:13:51 +0200 (CEST) Received: from debian.lan (unknown [IPv6:2a00:66c0:a::72c]) (Authenticated sender: git01) by glom.nmugroup.com (Postfix) with ESMTPSA id B32735429274 for ; Thu, 30 May 2024 00:13:48 +0200 (CEST) Message-ID: <8cfb78ebf68076f26aee97f951fc135e6aede100.camel@haerdin.se> From: Tomas =?iso-8859-1?q?H=E4rdin?= To: FFmpeg development discussions and patches Date: Thu, 30 May 2024 00:13:48 +0200 In-Reply-To: <27e5f614496e729f68326a31c3ac70d5923cfa97.camel@haerdin.se> References: <27e5f614496e729f68326a31c3ac70d5923cfa97.camel@haerdin.se> User-Agent: Evolution 3.46.4-2 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 2/5] lavu/common.h: Fix UB in av_clip_intp2_c() X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: gXVW0RJsW/KJ From 7b18f24c0bedfeebcdfb23ea837cea8d4c35cf30 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= Date: Thu, 16 May 2024 16:33:44 +0200 Subject: [PATCH 2/5] lavu/common.h: Fix UB in av_clip_intp2_c() Found by value analysis --- libavutil/common.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/common.h b/libavutil/common.h index ac68c0cfff..715f0a594c 100644 --- a/libavutil/common.h +++ b/libavutil/common.h @@ -264,7 +264,7 @@ static av_always_inline av_const int32_t av_clipl_int32_c(int64_t a) */ static av_always_inline av_const int av_clip_intp2_c(int a, int p) { - if (((unsigned)a + (1 << p)) & ~((2 << p) - 1)) + if (((unsigned)a + (1U << p)) & ~((2U << p) - 1)) return (a >> 31) ^ ((1 << p) - 1); else return a; -- 2.39.2