** Tags removed: verification-needed verification-needed-jammy verification-needed-noble ** Tags added: verification-done verification-done-jammy verification-done-noble
-- You received this bug notification because you are a member of Desktop Packages, which is subscribed to adsys in Ubuntu. https://bugs.launchpad.net/bugs/2078247 Title: rewrite from / to @ formats overwrites usernames Status in adsys package in Ubuntu: Fix Released Status in adsys source package in Jammy: Fix Committed Status in adsys source package in Noble: Fix Committed Bug description: tracking bug https://bugs.launchpad.net/ubuntu/+source/adsys/+bug/2078245 [ Impact ] In this code, the string was not zero terminated properly when overwritten resulting in extra characters: static char *slash_to_at_username(const char *username) { char *backslash = strchr(username, '\\'); if (backslash != NULL) { char *ret = malloc((strlen(username) + 1) * sizeof(char)); strcpy(ret, backslash + 1); strcat(ret, "@"); strncpy(ret + strlen(ret), username, backslash - username); return ret; } return strdup(username); For instance: domain.tld\\user was changed to [email protected] So se user cannot login with the domain\user format [ Test Plan ] 1. On a machine joined to an AD domain with ADsys installed, login with domain\user => Without the patch the user cannot log in => With the patch the user can log in [ Where problems could occur ] Worst case the user still cannot log in To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/adsys/+bug/2078247/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : [email protected] Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
