Code review comment for ~ahasenack/ubuntu/+source/freeradius:freeradius-3.0.26git-update

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Since those commits were on rlm_mschap, I tested mschap auth with radtest. I thought I had done it before, but maybe not (there are many authentication mechanisms).

Turns out mschap is core dumping with this 3.0.26 snapshot and openssl3 in jammy. With 3.0.21 and libssl1.1, it works. Can't tell yet if the problem is 3.0.26, or openssl3, or a combination.

I'll file a bug upstream, and one in LP, but I think we should proceed with this upload. What's more urgent now is to add DEP8 tests, and I will file a bug to at least have these simple radtest tests I'm doing converted into DEP8. I filed https://bugs.launchpad.net/ubuntu/+source/freeradius/+bug/1962046 to improve DEP8 coverage

« Back to merge proposal