Subject: Re: bug#61504: 29.0.60; executing byte-code from previous build causes SIGSEGV crash

From: Mattias Engdegård <[email-masked]>
Date: Tue, 14 Feb 2023 18:21:28 +0100

14 feb. 2023 kl. 18.00 skrev Gregory Heytings <[email-masked]>:

>> By the way, doesn't the patch switch the restoration order of narrowing and restriction, respectively? Maybe it doesn't matter?
>> 
> 
> Hmmm, that's a good question!  The evaluation order of parameters is unspecified in C, so actually the order could be switched or not, depending on what the compiler chooses to do.

Yes, the saving order is undefined but the restoring order seems well-defined. It currently restores narrowing locks first, then the restriction, but your patch flips the order.

Please at least make the saving order well-defined, preferably in the reverse order of restoration for symmetry.