27 Jun
2006
27 Jun
'06
1:19 p.m.
On 178, 06 27, 2006 at 11:07:51 +0400, Andrey Panin wrote:
On 177, 06 26, 2006 at 04:26:23PM +0300, Timo Sirainen wrote:
On Mon, 2006-06-26 at 16:58 +0400, Andrey Panin wrote:
- MEMBER(update_query) "UPDATE users SET password = '%c' WHERE userid = '%u'", ..
- buf = buffer_create_dynamic(unsafe_data_stack_pool, 128);
- tmp.key = 'c';
- tmp.value = new_credentials;
- buffer_append(buf, &tmp, sizeof(tmp));
- tab = auth_request_get_var_expand_table(request, passdb_sql_escape);
- do {
buffer_append(buf, tab++, sizeof(*tab));- } while (tab->key != '\0');
I haven't looked at the patches too closely yet, but how about using the existing %w instead of adding a new %c?
Good idea, but needs documentation update since %w is defined as "plaintext password from plaintext authentication mechanism" now.
Otherwise looks good. Well, except PASSS could be changed to something else :) SETCRED maybe.
As you wish :) I'll send an updated patches soon.
Updated patches attached, s/%c/%w/ and s/PASSS/SETCRED/
-- Andrey Panin | Linux and UNIX system administrator pazke@donpac.ru | PGP key: wwwkeys.pgp.net