~ecryptfs/ecryptfs/trunk

« back to all changes in this revision

Viewing changes to src/utils/umount.ecryptfs.c

  • Committer: Dustin Kirkland
  • Date: 2009-04-21 23:20:27 UTC
  • mfrom: (376.1.10 ecryptfs)
  • Revision ID: kirkland@canonical.com-20090421232027-xhtpj31xtf4c1n79
Merge from Michal Hlavinka.


386. By Michal Hlavinka 2 hours ago

    change error codes to be more descriptive

    Error codes were changed to be more descriptive. MOUNT_ERROR after
    asprintf changed to ENOMEM, some error codes are no longer filtered
    to MOUNT_ERROR. And some other changes mostly from general MOUNT_ERROR
    to ESOMETHING.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
385. By Michal Hlavinka 3 hours ago

    decision_graph.h, *: change definition of node return codes to positive values

    Using negative node return codes and negative error codes brings
    too big complications everywhere. Use positive values for node return codes.
    Change error codes that were changed to positive values back to negative
    values.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
384. By Michal Hlavinka 3 hours ago

    mount.ecryptfs.c: insist for yes/no answer for unkown sigs

    Mounting with new uknown sig is important question. Insist on yes/no
    answer. Don't assume everything else than yes is no.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
383. By Michal Hlavinka 4 hours ago

    don't print error for removing key from keyring if it succeeded

    Prevent printing of "Failed to remove ... with sig [...] from keyring: Success"

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
382. By Michal Hlavinka 4 hours ago

    module_mgr.c: insist on yes/no answer

    Insist on yes/no/y/n answer. Don't assume everything starting 'y'
    is yes and everything else is no. If wrong anser, return
    WRONG_VALUE for asking again.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
381. By Michal Hlavinka 4 hours ago

    use ECRYPTFS_NONEMPTY_VALUE_REQUIRED where reasonable

    Some empty values makes no sense. Require nonempty values
    in that case (for interactive mount).

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
380. By Michal Hlavinka 5 hours ago

    pam_ecryptfs.c: don't try to unwrap key for users not using pam mounting

    Don't delay passphrase verification for users not using pam automounting.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
379. By Michal Hlavinka 5 hours ago

    add verbosity to man page

    Added notes about verbosity to ecryptfs man page.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
378. By Michal Hlavinka 5 hours ago

    decision_graph.* : add ECRYPTFS_NONEMPTY_VALUE_REQUIRED flag for nodes

    Nodes can prevent future problems asking for non-empty value.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>
377. By Michal Hlavinka 11 hours ago

    decision_graph.* : add WRONG_VALUE return code to nodes for asking question again

    Failing mount because of typo is not nice. If node transaction function returns
    WRONG_VALUE and we can ask questions (verbosity=0 is not specified),
    ask question again.

    Signed-off-by: Michal Hlavinka <mhlavink@redhat.com>


Acked-by: Dustin Kirkland <kirkland@canonical.com>

Show diffs side-by-side

added added

removed removed

Lines of Context:
109
109
                fekek_fail = ecryptfs_remove_auth_tok_from_keyring(fekek_sig);
110
110
                if (fekek_fail)
111
111
                        fprintf(stderr, "Failed to remove fekek with sig [%s] "
112
 
                                "from keyring: %s\n", fekek_sig, strerror(rc));
 
112
                                "from keyring: %s\n", fekek_sig,
 
113
                                strerror(fekek_fail));
113
114
        } else {
114
115
                fekek_fail = rc;
115
116
        }
117
118
                                 "ecryptfs_fnek_sig=", &fnek_sig)
118
119
            && strcmp(fekek_sig, fnek_sig)) {
119
120
                fnek_fail = ecryptfs_remove_auth_tok_from_keyring(fnek_sig);
120
 
                if (fekek_fail) {
 
121
                if (fnek_fail) {
121
122
                        fprintf(stderr, "Failed to remove fnek with sig [%s] "
122
 
                                "from keyring: %s\n", fekek_sig, strerror(rc));
 
123
                                "from keyring: %s\n", fnek_sig, 
 
124
                                strerror(fnek_fail));
123
125
                }
124
126
        }
125
127
        free(fekek_sig);