On 03/08, Lucas De Marchi wrote: > > static int call_usermodehelper_keys(char *path, char **argv, char **envp, > struct key *session_keyring, int wait) > { > - return call_usermodehelper_fns(path, argv, envp, wait, > - umh_keys_init, umh_keys_cleanup, > - key_get(session_keyring)); > + struct subprocess_info *info; > + > + info = call_usermodehelper_setup(path, argv, envp, GFP_KERNEL, > + umh_keys_init, umh_keys_cleanup, > + key_get(session_keyring)); > + if (!info) { > + key_put(session_keyring); > + return -ENOMEM; > + } > + > + return call_usermodehelper_exec(info, wait);
Looks correct, but can't we simpluify it a bit? info = call_usermodehelper_setup(session_keyring); if (!info) return ENOMEM; key_get(session_keyring)); return call_usermodehelper_exec(info); Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/