Jozsef Kadlecsik via Postfix-users:
> +    sasl_username = xsasl_server_get_username(state->sasl_server);
> +    if (sasl_username != 0) {
> +     state->sasl_username = mystrdup(sasl_username);
> +     printable(state->sasl_username, '?');

1) There is no corresponding myfree() call. 

2) There is no need to 'censor' sasl_username if the name is valid.

3) In fact the msg() formatter already seems to censor logging;
there is some redundancy in Postfix code.

I think I can take it from here.

        Wietse
_______________________________________________
Postfix-users mailing list -- postfix-users@postfix.org
To unsubscribe send an email to postfix-users-le...@postfix.org

Reply via email to