On 05/07/2012 02:12 PM, Jakub Hrozek wrote:
I only have two comments related to code style:
+ TALLOC_CTX *memctx;
+ int n_etype_list;
+
+ memctx = talloc_new (NULL);
We usually call such memory context tmp_ctx so that it's clear it's a
short-lived context used only inside this function.
+ krb5_get_init_creds_opt_set_etype_list (&options, etype_list,
^^^
We don't use a space between a function name and the arguments list.
Thanks. Fixed. Attached.
Stef