From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id C6C58489D1 for ; Fri, 22 Dec 2023 12:15:56 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 95B3268D2EC; Fri, 22 Dec 2023 14:15:54 +0200 (EET) Received: from mail8.parnet.fi (mail8.parnet.fi [77.234.108.134]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id CEB1968CEE7 for ; Fri, 22 Dec 2023 14:15:47 +0200 (EET) Received: from mail9.parnet.fi (mail9.parnet.fi [77.234.108.21]) by mail8.parnet.fi with ESMTP id 3BMCFkUY025784-3BMCFkUZ025784; Fri, 22 Dec 2023 14:15:46 +0200 Received: from foo.martin.st (host-97-144.parnet.fi [77.234.97.144]) by mail9.parnet.fi (Postfix) with ESMTPS id 75419A146B; Fri, 22 Dec 2023 14:15:46 +0200 (EET) Date: Fri, 22 Dec 2023 14:15:45 +0200 (EET) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: FFmpeg development discussions and patches In-Reply-To: <20231222115959.362902-1-post@frankplowman.com> Message-ID: References: <20231222115959.362902-1-post@frankplowman.com> MIME-Version: 1.0 X-FE-Policy-ID: 3:14:2:SYSTEM Subject: Re: [FFmpeg-devel] [PATCH] avformat/ffrtmpcrypt: Fix int-conversion warning 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 Cc: Frank Plowman Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Hi Frank, On Fri, 22 Dec 2023, Frank Plowman wrote: > The gcrypt definition of `bn_new` used to use `AVERROR`, however it is > called in `dh_generate_key` and `ff_dh_init` which return pointers. As a > result, compiling with gcrypt and the ffrtmpcrypt protocol resulted in an > int-conversion warning. GCC 14 may upgrade these to errors [1]. (FWIW, the issue that bn_new was used in functions that don't return an error-signaling integer was present when this macro was added originally in d50b5d547f4070678c88aa095b5292c872e2c1dc to.) The change LGTM, but the wording here is slightly confusing IMO. The problem isn't with using per se AVERROR, that's just a macro for generating suitable integers, the issue is more about the fact that we're returning from a macro, without knowing the actual context where the macro is invoked. WDYT about this wording? > The gcrypt definition of `bn_new` used to use the return statement on > errors, with an AVERROR return value, regardless of the signature of the > function where the macro is used - it is called in `dh_generate_key` and > `ff_dh_init` which return pointers. As a result, compiling with gcrypt > and the ffrtmpcrypt protocol resulted in an int-conversion warning. GCC > 14 may upgrade these to errors [1]. // Martin _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".