Hi Adrian, On Sun, Nov 19, 2023 at 11:10:04PM +0200, Adrian Bunk wrote: > On Thu, Oct 05, 2023 at 09:38:00PM +0200, Salvatore Bonaccorso wrote: > > Hi Otto, > > > > Thanks for the quick followup. > > > > On Wed, Oct 04, 2023 at 08:59:31PM -0700, Otto Kekäläinen wrote: > > > Thanks for reporting this Salvatore! > > > > > > Are you aware of what plans upstream has? > > > > We are not, basically we require your help for this report for > > assessing the issue. > > > > > The Jira MDEV-25068 was fixed in Galera 26.4.12 > > > (https://releases.galeracluster.com/galera-4.12/release-notes-galera-26.4.12.txt) > > > in 2022. i don't see any commits on > > > https://github.com/codership/galera/commits/3.x since 2022. i will > > > keep an eye for new upstream releases. > > > > > > I can also review/merge for all Debian and Ubuntu releases still in > > > maintenance a patch if somebody wants to submit a Debian-specific fix > > > at https://salsa.debian.org/mariadb-team/galera-3/-/merge_requests. On > > > a quick look I did not find the 26.4.12 fix > > > (https://github.com/search?q=repo%3Acodership%2Fgalera+MDEV-25068&type=commits) > > > so I am not aware of any specific commit nor if it can be backported > > > to 25.3.37 > > > > Do you have a good upstream contact which you could reach out to ask > > on more details, references to fixes, etc on the issue? > > I looked at it for LTS and galera-3 is not affected: > > The upstream fix for MDEV-25068 is > https://github.com/codership/galera/commit/930c016108d7086b472ad7a8b9d0f6989202b48a > (26.4.12) > > This is in code that was introduced in > https://github.com/codership/galera/commit/c27596d06a221f6c14d36759c681149964008749 > (26.4.8) which was not backported to galera-3. > > The introducing commit merged assign_local_addr() and assign_remote_addr() > into assign_addresses(). > > The fix is to catch the error when assign_addresses() throws > asio::system_error. > > The two callsites of assign_local_addr/assign_remote_addr in the old code > in gcomm/src/asio_tcp.cpp are already (in 26.4.7 and 25.3.37): > try > { > ... > assign_local_addr() > assign_remote_addr() > ... > } > catch (asio::system_error& e) > { > ... > }
Thanks for the analysis of the issue. Regards, Salvatore