Port wincred keyring to PurpleCredentialManager

Review Request #466 — Created Jan. 30, 2021 and submitted

QuLogic
pidgin/pidgin
default
PIDGIN-17489
pidgin
  • Inline PURPLE_PLUGIN_INIT in wincred.
  • Add skeleton of wincred PurpleCredentialProvider object.
  • Implement password reading.
  • Implement wincred password writing.
  • Implement wincred password clearing.
  • Autoload wincred keyring plugin.
  • Cleanup formatting.

Added XMPP account, saved password, disabled and re-enabled account which kept the password. Restarted Pidgin and password was still saved. Removed XMPP account, and password was deleted from Credential Manager.

I'm not sure that the password needs to be converted from UTF-8 to "Unicode", as I did not test modifying the password in Credential Manager.

  • 0
  • 0
  • 2
  • 0
  • 2
Description From Last Updated
QuLogic
grim
QuLogic
grim
  1. 
      
  2. libpurple/plugins/keyrings/wincred.c (Diff revision 1)
     
     

    We should probably still clear this for security concerns as other programs could read it after the memory is freed back to the operating system.

  3. libpurple/plugins/keyrings/wincred.c (Diff revision 1)
     
     

    we should probably move purple_utf16_wipe to here (i believe it's the only use) or memset it to make sure it's nuked from memory.

  4. 
      
QuLogic
grim
  1. Ship It!
  2. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...