Looks good, except: > + if (reason == 0x00010002) can you add enums for all these SRP_LOGIN_REJ reason codes rather than open-coding this magic number here? Thanks, Roland