On Tue, 23 Sept 2025 at 09:31, Tomas Vondra <[email protected]> wrote: > On 9/22/25 22:45, David Rowley wrote: > > I think a1b4f289b mistakenly thought that there'd be size_t arithmetic > > in the following two lines because the final result is a size_t: > > > > size_t current_space = hash_table_bytes + (2 * nbatch * BLCKSZ); > > size_t new_space = hash_table_bytes * 2 + (nbatch * BLCKSZ); > > > > Yeah, I failed to notice this part of the formula can overflow.
Ok cool. We're just in the freeze for 18.0 at the moment. Once that's over, should I take care of this, or do you want to? David
