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 54C2F4A469 for ; Thu, 28 Mar 2024 11:14:07 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 9276468D671; Thu, 28 Mar 2024 13:14:05 +0200 (EET) Received: from glom.nmugroup.com (glom.nmugroup.com [193.183.80.6]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7E11F68BF41 for ; Thu, 28 Mar 2024 13:13:58 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by glom.nmugroup.com (Postfix) with ESMTP id DEC875429D1F for ; Thu, 28 Mar 2024 12:13:57 +0100 (CET) Received: from debian.lan (unknown [IPv6:2a00:66c0:a::72c]) (Authenticated sender: git01) by glom.nmugroup.com (Postfix) with ESMTPSA id 369945429D7C for ; Thu, 28 Mar 2024 12:13:56 +0100 (CET) Message-ID: From: Tomas =?ISO-8859-1?Q?H=E4rdin?= To: FFmpeg development discussions and patches Date: Thu, 28 Mar 2024 12:13:52 +0100 In-Reply-To: <20240328024856.GN6420@pb2> References: <20240320025923.3794-1-michael@niedermayer.cc> <092a8cd44e3074e2d0462052e715281c08eb31b9.camel@haerdin.se> <20240320131216.GR6420@pb2> <0e1355913d2f11aa14a712e1ea9a96781c10d290.camel@haerdin.se> <8f924f3320dd21fd5d41872c46566c6003f392bd.camel@haerdin.se> <20240325191313.GO6420@pb2> <20240325200424.GQ6420@pb2> <0892db70a5af069d8f02e50f3994ed8ebe886c5d.camel@haerdin.se> <20240328024856.GN6420@pb2> User-Agent: Evolution 3.46.4-2 MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH 1/4] avcodec/jpeg2000htdec: Check M_b / magp before using it in a shift 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 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: dG9yIDIwMjQtMDMtMjgga2xvY2thbiAwMzo0OCArMDEwMCBza3JldiBNaWNoYWVsIE5pZWRlcm1h eWVyOgo+IE9uIFdlZCwgTWFyIDI3LCAyMDI0IGF0IDExOjEzOjQ4QU0gKzAxMDAsIFRvbWFzIEjD pHJkaW4gd3JvdGU6Cj4gPiBtw6VuIDIwMjQtMDMtMjUga2xvY2thbiAyMTowNCArMDEwMCBza3Jl diBNaWNoYWVsIE5pZWRlcm1heWVyOgo+ID4gPiBPbiBNb24sIE1hciAyNSwgMjAyNCBhdCAwODox MzoxM1BNICswMTAwLCBNaWNoYWVsIE5pZWRlcm1heWVyCj4gPiA+IHdyb3RlOgo+ID4gPiA+IE9u IFRodSwgTWFyIDIxLCAyMDI0IGF0IDA0OjA3OjE0UE0gKzAxMDAsIFRvbWFzIEjDpHJkaW4gd3Jv dGU6Cj4gPiA+ID4gPiBvbnMgMjAyNC0wMy0yMCBrbG9ja2FuIDIxOjM1ICswMTAwIHNrcmV2IFRv bWFzIEjDpHJkaW46Cj4gPiA+ID4gPiA+IG9ucyAyMDI0LTAzLTIwIGtsb2NrYW4gMTQ6MTIgKzAx MDAgc2tyZXYgTWljaGFlbAo+ID4gPiA+ID4gPiBOaWVkZXJtYXllcjoKPiA+ID4gPiA+ID4gPiBP biBXZWQsIE1hciAyMCwgMjAyNCBhdCAxMjoyMDoxMVBNICswMTAwLCBUb21hcyBIw6RyZGluCj4g PiA+ID4gPiA+ID4gd3JvdGU6Cj4gPiA+ID4gPiA+ID4gPiBvbnMgMjAyNC0wMy0yMCBrbG9ja2Fu IDAzOjU5ICswMTAwIHNrcmV2IE1pY2hhZWwKPiA+ID4gPiA+ID4gPiA+IE5pZWRlcm1heWVyOgo+ ID4gPiA+ID4gPiA+ID4gPiBGaXhlczogc2hpZnQgZXhwb25lbnQgLTEgaXMgbmVnYXRpdmUKPiA+ ID4gPiA+ID4gPiA+ID4gRml4ZXM6IDY1Mzc4L2NsdXN0ZXJmdXp6LXRlc3RjYXNlLW1pbmltaXpl ZC0KPiA+ID4gPiA+ID4gPiA+ID4gZmZtcGVnX0FWX0NPREVDX0lEX0pQRUcyMDAwX2Z1enplci01 NDU3Njc4MTkzMTk3MDU2Cj4gPiA+ID4gPiA+ID4gPiA+IAo+ID4gPiA+ID4gPiA+ID4gPiBGb3Vu ZC1ieTogY29udGludW91cyBmdXp6aW5nIHByb2Nlc3MKPiA+ID4gPiA+ID4gPiA+ID4gaHR0cHM6 Ly9naXRodWIuY29tL2dvb2dsZS9vc3MtZnV6ei90cmVlL21hc3Rlci9wcm9qZWN0cy9mZm1wZWcK PiA+ID4gPiA+ID4gPiA+ID4gU2lnbmVkLW9mZi1ieTogTWljaGFlbCBOaWVkZXJtYXllcgo+ID4g PiA+ID4gPiA+ID4gPiA8bWljaGFlbEBuaWVkZXJtYXllci5jYz4KPiA+ID4gPiA+ID4gPiA+ID4g LS0tCj4gPiA+ID4gPiA+ID4gPiA+IMKgbGliYXZjb2RlYy9qcGVnMjAwMGh0ZGVjLmMgfCAzICsr Kwo+ID4gPiA+ID4gPiA+ID4gPiDCoDEgZmlsZSBjaGFuZ2VkLCAzIGluc2VydGlvbnMoKykKPiA+ ID4gPiA+ID4gPiA+ID4gCj4gPiA+ID4gPiA+ID4gPiA+IGRpZmYgLS1naXQgYS9saWJhdmNvZGVj L2pwZWcyMDAwaHRkZWMuYwo+ID4gPiA+ID4gPiA+ID4gPiBiL2xpYmF2Y29kZWMvanBlZzIwMDBo dGRlYy5jCj4gPiA+ID4gPiA+ID4gPiA+IGluZGV4IDZiOTg5OGQzZmYuLjBiOTRiYjVkYTIgMTAw NjQ0Cj4gPiA+ID4gPiA+ID4gPiA+IC0tLSBhL2xpYmF2Y29kZWMvanBlZzIwMDBodGRlYy5jCj4g PiA+ID4gPiA+ID4gPiA+ICsrKyBiL2xpYmF2Y29kZWMvanBlZzIwMDBodGRlYy5jCj4gPiA+ID4g PiA+ID4gPiA+IEBAIC0xMTkzLDYgKzExOTMsOSBAQCBmZl9qcGVnMjAwMF9kZWNvZGVfaHRqMmso Y29uc3QKPiA+ID4gPiA+ID4gPiA+ID4gSnBlZzIwMDBEZWNvZGVyQ29udGV4dCAqcywgSnBlZzIw MDBDb2RpbmdTdHlsZSAqYwo+ID4gPiA+ID4gPiA+ID4gPiDCoAo+ID4gPiA+ID4gPiA+ID4gPiDC oMKgwqDCoCBpbnQzMl90IE1fYiA9IG1hZ3A7Cj4gPiA+ID4gPiA+ID4gPiA+IMKgCj4gPiA+ID4g PiA+ID4gPiA+ICvCoMKgwqAgaWYgKG1hZ3AgPj0gMzEpCj4gPiA+ID4gPiA+ID4gPiA+ICvCoMKg wqDCoMKgwqDCoCByZXR1cm4gQVZFUlJPUl9JTlZBTElEREFUQTsKPiA+ID4gPiA+ID4gPiA+IAo+ ID4gPiA+ID4gPiA+ID4gVGhpcyBpc24ndCB3aGVyZSB0aGUgZXJyb3IgaXMsIGFzc3VtaW5nIGl0 IGV2ZW4gaXMgYW4KPiA+ID4gPiA+ID4gPiA+IGVycm9yLiBJdCdzCj4gPiA+ID4gPiA+ID4gPiBl aXRoZXIgZXhwbiBvciBuZ3VhcmRiaXRzIHRoYXQgYXJlIHdyb25nLCBhbmQgdGhleQo+ID4gPiA+ ID4gPiA+ID4gc2hvdWxkCj4gPiA+ID4gPiA+ID4gPiBiZQo+ID4gPiA+ID4gPiA+ID4gZGV0ZWN0 ZWQKPiA+ID4gPiA+ID4gPiA+IGFuZCByZXBvcnRlZCBhcyBzdWNoIGluIGpwZWcyMDAwZGVjLmMu IENoZWNraW5nIHRoaXMgaW4KPiA+ID4gPiA+ID4gPiA+IGV2ZXJ5Cj4gPiA+ID4gPiA+ID4gPiBj YWxsCj4gPiA+ID4gPiA+ID4gPiB0bwo+ID4gPiA+ID4gPiA+ID4gZmZfanBlZzIwMDBfZGVjb2Rl X2h0ajJrKCkgaXMgd2FzdGVmdWwuCj4gPiA+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4gPiA+IG5n dWFyZGJpdHMgY2FuIGJlIDAuLjcgYW5kIGV4cG4gY2FuIGJlIDAuLjMxLiBUYWJsZQo+ID4gPiA+ ID4gPiA+ID4gQS4xMQo+ID4gPiA+ID4gPiA+ID4gaW5kaWNhdGVzCj4gPiA+ID4gPiA+ID4gPiB0 aGF0Cj4gPiA+ID4gPiA+ID4gPiBTc2l6ZSBjYW4gYmUgdXAgdG8gMzggYml0cywgc28gTV9iID49 IDMxIGlzIGluIGZhY3QKPiA+ID4gPiA+ID4gPiA+IHBlcmZlY3RseQo+ID4gPiA+ID4gPiA+ID4g dmFsaWQuCj4gPiA+ID4gPiA+ID4gCj4gPiA+ID4gPiA+ID4gPiBBCj4gPiA+ID4gPiA+ID4gPiBt b3JlIGFwcHJvcHJpYXRlIGVycm9yIG1pZ2h0IGJlIEFWRVJST1JfUEFUQ0hXRUxDT01FLgo+ID4g PiA+ID4gPiA+IAo+ID4gPiA+ID4gPiA+IGluZGVlZCwgaSB3aWxsIGNoYW5nZSBpdCB0byBBVkVS Uk9SX1BBVENIV0VMQ09NRQo+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4gUGxlYXNlIGFsc28gbW92 ZSBpdCBmdXJ0aGVyIHVwIHNvIGFzIHRvIG5vdCB3YXN0ZSBjeWNsZXMKPiA+ID4gPiA+ID4gY2hl Y2tpbmcgaXQKPiA+ID4gPiA+ID4gZXZlcnkgdGltZQo+ID4gPiA+ID4gCj4gPiA+ID4gPiBUbyBi ZSBtb3JlIHByZWNpc2UsIGdldF9xY3goKSBsb29rcyBsaWtlIHRoZSBwcm9wZXIgcGxhY2UgZm9y Cj4gPiA+ID4gPiBpdAo+ID4gPiA+IAo+ID4gPiA+IHdpbGwgYXBwbHkgd2l0aCB0ZWggY2hlY2sg bW92ZWQgdGhlcmUKPiA+ID4gCj4gPiA+IHRoZSB2YWx1ZXMgdGhhdCBhcmUgY2F1c2luZyB1bmRl ZmluZWQgYmVoYXZpb3IgZm9yIGh0ajJrIGFyZSB1c2VkCj4gPiA+IGluCj4gPiA+IG5vcm1hbAo+ ID4gPiBqMmsga25vd2luZyB3aGljaCB0eXBlIG9mIGoyayB3ZSBoYXZlIHNlZW1zIGRlY2lkZWQg YnkKPiA+ID4gQ09DL0NPRC9DT1gKPiA+ID4gCj4gPiA+IHNvIGkgZG9udCB0aGluayB3ZSBjYW4g Y2hlY2sgaW4gUUNYLCBiZWNhdXNlIGEgbGF0ZXIgQ09YIGNvdWxkCj4gPiA+IG1ha2UgaXQgYm90 aCBpbnZhbGlkIG9yIHZhbGlkCj4gPiA+IGFuZCB3ZSBjYW5ub3QgY2hlY2sgaW4gQ09YIGFzIGEg bGF0ZXIgUUNYIGNhbiBzaW1pbGFybHkgY2hhbmdlIGl0Cj4gPiAKPiA+IFRoYXQgYWxsIGNhbGxz IGdldF9xY3goKS4KPiAKPiB5ZXMKPiAKPiAKPiA+IElmIHlvdSBnaXQgZ3JlcCBmb3Igbmd1YXJk Yml0cyB5b3UnbGwgc2VlCj4gPiBpdCdzIG9ubHkgZXZlciBzZXQgdGhlcmUgd2hlbiBkZWNvZGlu ZywgYW5kIHNpbWlsYXJseSB3aXRoIGV4cG4uCj4gCj4geWVzCj4gCj4gCj4gPiBDb2RpbmcKPiA+ IHN0eWxlIGFuZCBxdWFudGl6YXRpb24gc3R5bGUgYXJlIG5vdCB0aGUgc2FtZSB0aGluZy4KPiAK PiB5ZXMKPiAKPiAKPiBidXQgc3RpbGwsIHlvdSBjYW4gdHJ5IHRvIGFkZCBhIGNoZWNrIHRoZSB2 YWx1ZXMgZm9yIGJvdGggbmd1YXJkYml0cwo+IGFuZCBleHBuIHdoaWNoIGxlYWQgdG8gdW5kZWZp bmVkIHNoaWZ0cyBpbiBmZl9qcGVnMjAwMF9kZWNvZGVfaHRqMmsoKQo+IGFyZSB1c2VkIGluIG5v cm1hbCBqcGVnMjAwMCBhbmQgYnJlYWsgdGhlc2Ugc2FtcGxlcwoKSHVtIGh1bS4uIFlvdSBzZWVt IHRvIGJlIHNwZWFraW5nIG9mIGNibGstPm5vbnplcm9iaXRzIHdoaWNoIGlzIHVzZWQgaW4KZGVj b2RlX2NibGsoKSBmb3IgdGhlIFBhcnQgMSBkZWNvZGVyLiBJdCBvbmx5IGFjY2VwdHMgYnBubyBp biAwLi4yOS4KYnBubyBpbiB0dXJuIGRlcGVuZHMgb24gcm9pX3NoaWZ0IHdoaWNoIGlzIG5ldmVy IG5lZ2F0aXZlLgoKICBicG5vID0gZXhwbiArIG5ndWFyZGJpdHMgKyByb2lfc2hpZnQgLSAxIC0g emJwCgpCdXQgemJwIGNhbiBiZSB1cCB0byAxMDAgaWYgSSB1bmRlcnN0YW5kIGNvcnJlY3RseSwg c28gZXhwbiArIGd1YXJkYml0cwpjYW4gYmUgdXAgdG8gMTMwIGFuZCBzdGlsbCBiZSBsZWdhbCBQ YXJ0IDEuIFNvIHllYWgsIHB1dHRpbmcgdGhlIGNoZWNrCmluIGdldF9xY3goKSBpc24ndCByaWdo dC4gQnV0IGl0IGNvdWxkIGxpdmUgaW4gdGlsZV9jb2RlYmxvY2tzKCkgaW4gdGhlCmJhbmRubyBs b29wLiBBcyBhIGJvbnVzIHdlIG9ubHkgbmVlZCB0byBjb21wdXRlIG1hZ3Agb25jZSBwZXIgc3Vi YmFuZAoodGhlIGNvbXBpbGVyIHByb2JhYmx5IGFscmVhZHkgbm90aWNlcyB0aGlzKS4KCkludGVy ZXN0aW5nbHkgcm9pX3NoaWZ0IGlzIHNlbnQgdG8gZmZfanBlZzIwMDBfZGVjb2RlX2h0ajJrKCkg YnV0IG5ldmVyCnVzZWQuIFQuODE0IHNlZW1zIHRvIGluZGljYXRlIFJPSSBpcyBzdGlsbCB1c2Vk IGZvciBIVEoySyBzbyBtYXliZSB3ZQpzaG91bGQgd2FybiBpZiBpdCdzIG5vbi16ZXJvPwoKL1Rv bWFzCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmZmbXBl Zy1kZXZlbCBtYWlsaW5nIGxpc3QKZmZtcGVnLWRldmVsQGZmbXBlZy5vcmcKaHR0cHM6Ly9mZm1w ZWcub3JnL21haWxtYW4vbGlzdGluZm8vZmZtcGVnLWRldmVsCgpUbyB1bnN1YnNjcmliZSwgdmlz aXQgbGluayBhYm92ZSwgb3IgZW1haWwKZmZtcGVnLWRldmVsLXJlcXVlc3RAZmZtcGVnLm9yZyB3 aXRoIHN1YmplY3QgInVuc3Vic2NyaWJlIi4K