From 6d1e1b5ddaab0ab6da8d5d14d73469ca5c4497ce Mon Sep 17 00:00:00 2001 From: ueno Date: Sat, 29 Apr 2006 21:04:03 +0000 Subject: [PATCH] * epg.el (epg-make-signature): Removed user-id; added creation-time, expiration-time, pubkey-algorithm, digest-algorithm. (epg-process-sentinel): Abolished. --- ChangeLog | 7 ++ epg.el | 333 +++++++++++++++++++++++++++++++++---------------------------- 2 files changed, 187 insertions(+), 153 deletions(-) diff --git a/ChangeLog b/ChangeLog index c5c3955..f0b649b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2006-04-29 Daiki Ueno + * epg.el (epg-make-signature): Removed user-id; added + creation-time, expiration-time, pubkey-algorithm, + digest-algorithm. + (epg-process-sentinel): Abolished. + +2006-04-29 Daiki Ueno + * epg.el (epg-status-ERRSIG): New function. (epg-verify-file): Signal an error if we encounter ERRSIG status. (epg-verify-string): Ditto. diff --git a/epg.el b/epg.el index 3a2a1c1..672a7a9 100644 --- a/epg.el +++ b/epg.el @@ -336,9 +336,9 @@ This function is for internal use only." (signal 'wrong-type-argument (list 'epg-context-p context))) (aset (cdr context) 12 result)) -(defun epg-make-signature (status key-id user-id) +(defun epg-make-signature (status &optional key-id) "Return a signature object." - (cons 'epg-signature (vector status key-id user-id nil nil))) + (cons 'epg-signature (vector status key-id nil nil nil nil nil nil))) (defun epg-signature-status (signature) "Return the status code of SIGNATURE." @@ -352,24 +352,42 @@ This function is for internal use only." (signal 'wrong-type-argument (list 'epg-signature-p signature))) (aref (cdr signature) 1)) -(defun epg-signature-user-id (signature) - "Return the user-id of SIGNATURE." - (unless (eq (car signature) 'epg-signature) - (signal 'wrong-type-argument (list 'epg-signature-p signature))) - (aref (cdr signature) 2)) - (defun epg-signature-validity (signature) "Return the validity of SIGNATURE." (unless (eq (car signature) 'epg-signature) (signal 'wrong-type-argument (list 'epg-signature-p signature))) - (aref (cdr signature) 3)) + (aref (cdr signature) 2)) (defun epg-signature-fingerprint (signature) "Return the fingerprint of SIGNATURE." (unless (eq (car signature) 'epg-signature) (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aref (cdr signature) 3)) + +(defun epg-signature-creation-time (signature) + "Return the creation time of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) (aref (cdr signature) 4)) +(defun epg-signature-expiration-time (signature) + "Return the expiration time of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aref (cdr signature) 5)) + +(defun epg-signature-pubkey-algorithm (signature) + "Return the public key algorithm of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aref (cdr signature) 6)) + +(defun epg-signature-digest-algorithm (signature) + "Return the digest algorithm of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aref (cdr signature) 7)) + (defun epg-signature-set-status (signature status) "Set the status code of SIGNATURE." (unless (eq (car signature) 'epg-signature) @@ -382,23 +400,41 @@ This function is for internal use only." (signal 'wrong-type-argument (list 'epg-signature-p signature))) (aset (cdr signature) 1 key-id)) -(defun epg-signature-set-user-id (signature user-id) - "Set the user-id of SIGNATURE." - (unless (eq (car signature) 'epg-signature) - (signal 'wrong-type-argument (list 'epg-signature-p signature))) - (aset (cdr signature) 2 user-id)) - (defun epg-signature-set-validity (signature validity) "Set the validity of SIGNATURE." (unless (eq (car signature) 'epg-signature) (signal 'wrong-type-argument (list 'epg-signature-p signature))) - (aset (cdr signature) 3 validity)) + (aset (cdr signature) 2 validity)) (defun epg-signature-set-fingerprint (signature fingerprint) "Set the fingerprint of SIGNATURE." (unless (eq (car signature) 'epg-signature) (signal 'wrong-type-argument (list 'epg-signature-p signature))) - (aset (cdr signature) 4 fingerprint)) + (aset (cdr signature) 3 fingerprint)) + +(defun epg-signature-set-creation-time (signature creation-time) + "Set the creation time of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aset (cdr signature) 4 creation-time)) + +(defun epg-signature-set-expiration-time (signature expiration-time) + "Set the expiration time of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aset (cdr signature) 5 expiration-time)) + +(defun epg-signature-set-pubkey-algorithm (signature pubkey-algorithm) + "Set the public key algorithm of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aset (cdr signature) 6 pubkey-algorithm)) + +(defun epg-signature-set-digest-algorithm (signature digest-algorithm) + "Set the digest algorithm of SIGNATURE." + (unless (eq (car signature) 'epg-signature) + (signal 'wrong-type-argument (list 'epg-signature-p signature))) + (aset (cdr signature) 7 digest-algorithm)) (defun epg-make-key (owner-trust) "Return a key object." @@ -541,14 +577,28 @@ This function is for internal use only." (epg-context-set-result context (cons (cons name value) result))))) (defun epg-signature-to-string (signature) - (format "%s signature from %s %s%s" - (capitalize (symbol-name (epg-signature-status signature))) - (epg-signature-key-id signature) - (epg-signature-user-id signature) - (if (epg-signature-validity signature) - (format " (trust %s)" - (epg-signature-validity signature)) - ""))) + (let ((user-id (cdr (assoc (epg-signature-key-id signature) + epg-user-id-alist)))) + (concat + (cond ((eq (epg-signature-status signature) 'good) + "Good signature ") + ((eq (epg-signature-status signature) 'bad) + "Bad signature ") + ((eq (epg-signature-status signature) 'expired) + "Expired signature ") + ((eq (epg-signature-status signature) 'expired-key) + "Signature made by expired key ") + ((eq (epg-signature-status signature) 'revoked-key) + "Signature made by revoked key ") + ((eq (epg-signature-status signature) 'no-pubkey) + "No public key for ")) + (epg-signature-key-id signature) + (if user-id + (concat " from " user-id " ") + "") + (if (epg-signature-validity signature) + (format "(trust %s)" (epg-signature-validity signature)) + "")))) (defun epg-verify-result-to-string (verify-result) (mapconcat #'epg-signature-to-string verify-result "\n")) @@ -601,7 +651,6 @@ This function is for internal use only." args))) (set-default-file-modes orig-mode)) (set-process-filter process #'epg-process-filter) - (set-process-sentinel process #'epg-process-sentinel) (epg-context-set-process context process))) (defun epg-process-filter (process input) @@ -633,21 +682,6 @@ This function is for internal use only." (forward-line)) (setq epg-read-point (point))))) -(defun epg-process-sentinel (process status) - (if (and (buffer-live-p (process-buffer process)) - (not (equal status "finished\n"))) - (save-excursion - (set-buffer (process-buffer process)) - ;; gpg process exited abnormally, but we have not received an - ;; error response from it. Set it here. - (unless (epg-context-result-for epg-context 'error) - (if (string-match "\\`exited abnormally with code \\(.*\\)\n" status) - (epg-context-set-result-for - epg-context 'error - (list (cons 'exit (string-to-number (match-string 1 status))))) - (epg-context-set-result-for epg-context 'error - (list (cons 'signal status)))))))) - (defun epg-read-output (context) (with-temp-buffer (if (fboundp 'set-buffer-multibyte) @@ -774,105 +808,107 @@ This function is for internal use only." (epg-context-result-for epg-context 'error))) (delete-process process))))) -(defun epg-status-GOODSIG (process string) +(defun epg-signature-status-internal (status string) (if (string-match "\\`\\([^ ]+\\) \\(.*\\)" string) - (epg-context-set-result-for - epg-context - 'verify - (cons (epg-make-signature - 'good - (match-string 1 string) - (if (eq (epg-context-protocol epg-context) 'CMS) - (condition-case nil - (epg-dn-from-string (match-string 2 string)) - (error (match-string 2 string))) - (match-string 2 string))) - (epg-context-result-for epg-context 'verify))))) + (let* ((key-id (match-string 1 string)) + (user-id (match-string 2 string)) + (entry (assoc key-id epg-user-id-alist))) + (epg-context-set-result-for + epg-context + 'verify + (cons (epg-make-signature status key-id) + (epg-context-result-for epg-context 'verify))) + (if (eq (epg-context-protocol epg-context) 'CMS) + (condition-case nil + (setq user-id (epg-dn-from-string user-id)) + (error))) + (if entry + (setcdr entry user-id) + (setq epg-user-id-alist + (cons (cons key-id user-id) epg-user-id-alist)))) + (epg-context-set-result-for + epg-context + 'verify + (cons (epg-make-signature status) + (epg-context-result-for epg-context 'verify))))) + +(defun epg-status-GOODSIG (process string) + (epg-signature-status-internal 'good string)) (defun epg-status-EXPSIG (process string) - (if (string-match "\\`\\([^ ]+\\) \\(.*\\)" string) - (epg-context-set-result-for - epg-context - 'verify - (cons (epg-make-signature - 'expired - (match-string 1 string) - (if (eq (epg-context-protocol epg-context) 'CMS) - (condition-case nil - (epg-dn-from-string (match-string 2 string)) - (error (match-string 2 string))) - (match-string 2 string))) - (epg-context-result-for epg-context 'verify))))) + (epg-signature-status-internal 'expired string)) (defun epg-status-EXPKEYSIG (process string) - (if (string-match "\\`\\([^ ]+\\) \\(.*\\)" string) - (epg-context-set-result-for - epg-context - 'verify - (cons (epg-make-signature - 'expired-key - (match-string 1 string) - (if (eq (epg-context-protocol epg-context) 'CMS) - (condition-case nil - (epg-dn-from-string (match-string 2 string)) - (error (match-string 2 string))) - (match-string 2 string))) - (epg-context-result-for epg-context 'verify))))) + (epg-signature-status-internal 'expired-key string)) (defun epg-status-REVKEYSIG (process string) - (if (string-match "\\`\\([^ ]+\\) \\(.*\\)" string) - (epg-context-set-result-for - epg-context - 'verify - (cons (epg-make-signature - 'revoked-key - (match-string 1 string) - (if (eq (epg-context-protocol epg-context) 'CMS) - (condition-case nil - (epg-dn-from-string (match-string 2 string)) - (error (match-string 2 string))) - (match-string 2 string))) - (epg-context-result-for epg-context 'verify))))) + (epg-signature-status-internal 'revoked-key string)) (defun epg-status-BADSIG (process string) - (if (string-match "\\`\\([^ ]+\\) \\(.*\\)" string) - (epg-context-set-result-for - epg-context - 'verify - (cons (epg-make-signature - 'bad - (match-string 1 string) - (if (eq (epg-context-protocol epg-context) 'CMS) - (condition-case nil - (epg-dn-from-string (match-string 2 string)) - (error (match-string 2 string))) - (match-string 2 string))) - (epg-context-result-for epg-context 'verify))))) + (epg-signature-status-internal 'bad string)) + +(defun epg-status-NO_PUBKEY (process string) + (epg-context-set-result-for + epg-context + 'verify + (cons (epg-make-signature 'no-pubkey string) + (epg-context-result-for epg-context 'verify)))) (defun epg-status-ERRSIG (process string) - (if (string-match "\\`\\([^ ]+\\) \\([0-9]+\\) \\([0-9]+\\) \ + (let ((signatures (car (epg-context-result-for epg-context 'verify)))) + (unless signatures + (setq signatures (list (epg-make-signature 'error))) + (epg-context-set-result-for epg-context 'verify signatures)) + (when (and (not (eq (epg-signature-status (car signatures)) 'good)) + (string-match "\\`\\([^ ]+\\) \\([0-9]+\\) \\([0-9]+\\) \ \\([0-9A-Fa-f][0-9A-Fa-f]\\) \\([^ ]+\\) \\([0-9]+\\)" - string) - (epg-context-set-result-for - epg-context - 'verify - (cons (list (cons 'key-id (match-string 1 string)) - (cons 'pubkey-algorithm - (string-to-number (match-string 2 string))) - (cons 'digest-algorithm - (string-to-number (match-string 3 string))) - (cons 'class - (string-to-number (match-string 4 string))) - (cons 'creation-time (match-string 5 string)) - (cons 'rc (string-to-number (match-string 6 string)))) - (epg-context-result-for epg-context 'error))))) + string)) + (epg-signature-set-key-id + (car signatures) + (match-string 1 string)) + (epg-signature-set-pubkey-algorithm + (car signatures) + (string-to-number (match-string 2 string))) + (epg-signature-set-digest-algorithm + (car signatures) + (string-to-number (match-string 3 string))) +; (epg-signature-set-class +; (car signatures) +; (string-to-number (match-string 4 string) 16)) + (epg-signature-set-creation-time + (car signatures) + (match-string 5 string))))) (defun epg-status-VALIDSIG (process string) (let ((signature (car (epg-context-result-for epg-context 'verify)))) - (if (and signature - (eq (epg-signature-status signature) 'good) - (string-match "\\`\\([^ ]+\\) " string)) - (epg-signature-set-fingerprint signature (match-string 1 string))))) + (when (and signature + (eq (epg-signature-status signature) 'good) + (string-match "\\`\\([^ ]+\\) [^ ]+ \\([^ ]+\\) \\([^ ]+\\) \ +\\([0-9]+\\) [^ ]+ \\([0-9]+\\) \\([0-9]+\\) \\([0-9A-Fa-f][0-9A-Fa-f]\\) \ +\\(.*\\)" + string)) + (epg-signature-set-fingerprint + signature + (match-string 1 string)) + (epg-signature-set-creation-time + signature + (match-string 2 string)) + (epg-signature-set-expiration-time + signature + (match-string 3 string)) +; (epg-signature-set-version +; signature +; (string-to-number (match-string 4 string))) + (epg-signature-set-pubkey-algorithm + signature + (string-to-number (match-string 5 string))) + (epg-signature-set-digest-algorithm + signature + (string-to-number (match-string 6 string))) +; (epg-signature-set-class +; signature +; (string-to-number (match-string 7 string) 16)) + ))) (defun epg-status-TRUST_UNDEFINED (process string) (let ((signature (car (epg-context-result-for epg-context 'verify)))) @@ -1297,9 +1333,9 @@ For a normal or a clear text signature, SIGNED-TEXT should be nil." (epg-start-verify context (epg-make-data-from-file signature))) (epg-wait-for-completion context) - (if (epg-context-result-for context 'error) - (error "Verify failed: %S" - (epg-context-result-for context 'error))) +; (if (epg-context-result-for context 'error) +; (error "Verify failed: %S" +; (epg-context-result-for context 'error))) (unless plain (epg-read-output context))) (unless plain @@ -1329,9 +1365,9 @@ For a normal or a clear text signature, SIGNED-TEXT should be nil." (epg-start-verify context (epg-make-data-from-string signature))) (epg-flush context) (epg-wait-for-completion context) - (if (epg-context-result-for context 'error) - (error "Verify failed: %S" - (epg-context-result-for context 'error))) +; (if (epg-context-result-for context 'error) +; (error "Verify failed: %S" +; (epg-context-result-for context 'error))) (epg-read-output context)) (epg-delete-output-file context) (if (and input-file @@ -1419,10 +1455,7 @@ Otherwise, it makes a normal signature." (epg-start-sign context (epg-make-data-from-string plain) mode) (epg-flush context) (epg-wait-for-completion context) - (if (epg-context-result-for context 'sign) - (if (epg-context-result-for context 'error) - (message "Sign warning: %S" - (epg-context-result-for context 'error))) + (unless (epg-context-result-for context 'sign) (if (epg-context-result-for context 'error) (error "Sign failed: %S" (epg-context-result-for context 'error)) @@ -1487,15 +1520,12 @@ If RECIPIENTS is nil, it performs symmetric encryption." (epg-start-encrypt context (epg-make-data-from-file plain) recipients sign always-trust) (epg-wait-for-completion context) - (if sign - (if (epg-context-result-for context 'sign) - (if (epg-context-result-for context 'error) - (message "Sign warning: %S" - (epg-context-result-for context 'error))) - (if (epg-context-result-for context 'error) - (error "Sign failed: %S" - (epg-context-result-for context 'error)) - (error "Sign failed")))) + (if (and sign + (not (epg-context-result-for context 'sign))) + (if (epg-context-result-for context 'error) + (error "Sign failed: %S" + (epg-context-result-for context 'error)) + (error "Sign failed"))) (if (epg-context-result-for context 'error) (error "Encrypt failed: %S" (epg-context-result-for context 'error))) @@ -1518,15 +1548,12 @@ If RECIPIENTS is nil, it performs symmetric encryption." recipients sign always-trust) (epg-flush context) (epg-wait-for-completion context) - (if sign - (if (epg-context-result-for context 'sign) - (if (epg-context-result-for context 'error) - (message "Sign warning: %S" - (epg-context-result-for context 'error))) - (if (epg-context-result-for context 'error) - (error "Sign failed: %S" - (epg-context-result-for context 'error)) - (error "Sign failed")))) + (if (and sign + (not (epg-context-result-for context 'sign))) + (if (epg-context-result-for context 'error) + (error "Sign failed: %S" + (epg-context-result-for context 'error)) + (error "Sign failed"))) (if (epg-context-result-for context 'error) (error "Encrypt failed: %S" (epg-context-result-for context 'error))) -- 1.7.10.4