I don't think that's the case, Tim.

The version of seahorse-nautilus in utopic, 3.8.0, is calling
"cryptui_prompt_recipients". This cryptui method consists of the
following call:

        return cryptui_prompt_recipients_with_symmetric (keyset, title,
signer, NULL);

So it's cryptui that is itself supplying the NULL value.

Further, the function comment you cite specifies that NULL is a valid
input when symmetric encryption is not desired. The comment around the
patch states "Ask for recipients if symmetric encryption was not
requested or if the user didn't give a symmetric passphrase." It seems
both appropriate and the intention of the code (by the comment) to pop
up a key chooser dialog if the caller requests that symmetric encryption
should not be used. However, if the caller ever passes a NULL then no
dialog will be popped up.

It does seem like a logic error to me.

Finally, I will note that the latest version of seahorse-nautilus that I
looked at in bzr does call cryptui_prompt_recipients_with_symmetric,
does supply a non-null value for "symmetric", and does seem to work
fine. However, it seems to me that callers to libcryptui's
"cryptui_prompt_recipients" won't get the results from the function that
they would expect.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1394582

Title:
  patch - fix logic flaw, make seahorse-tool's encrypt command work

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/libcryptui/+bug/1394582/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to