[PATCH] Clean up DP registration and PAM request interfaces

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

There is no need to pass "domain" as a parameter to the data provider.
We're already talking directly to the provider for that domain. Both the
provider and responder are fully aware of which domain is being queried
without passing this parameter. It only serves to complicate the interface.


- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCn3EACgkQeiVVYja6o6NncQCeJsCFmDW9wDebX+zYZUzeVbWt
> UV0AnA1vldKN74klO3MiHd2Aeq5mSzmF
> =N31B
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/22/2010 10:14 AM, Stephen Gallagher wrote:
>  There is no need to pass "domain" as a parameter to the
> data provider.
>  We're already talking directly to the provider for that domain. Both the
>  provider and responder are fully aware of which domain is being queried
>  without passing this parameter. It only serves to complicate the interface.
>  
>   
Simo noticed a typo on this patch during code review. Fixed.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCshsACgkQeiVVYja6o6OXsQCeIzKtHPSMaqwkfef0Z5RvN0iv
> EIoAnjeMj5PuHOcvUt8ktqx+flN7fXmn
> =L0uQ
> -----END PGP SIGNATURE-----

[Jakub Hrozek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/22/2010 05:34 PM, Stephen Gallagher wrote:
>  Simo noticed a typo on this patch during code review. Fixed. 
* providers/data_provider.h:67 still mentions "domains" in the PAM
response packet
* I'm not sure about removing pd->domain from pam_print_data(). It is
still used elsewhere in the code (via the DEBUG_PAM_DATA() macro)

Otherwise looks good to me.

Ack to the other (DP registration) patch.
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCvJYACgkQHsardTLnvCVDrwCgrn59GKd28kBPQk0jwYoQjWbO
> Y4AAoLXqGCnDtEGeyoML8JbFZTBaxsMO
> =w3BY
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/22/2010 12:19 PM, Jakub Hrozek wrote:
>  On 02/22/2010 05:34 PM, Stephen Gallagher wrote:
> > Simo noticed a typo on this patch during code review. Fixed.
>  
>  * providers/data_provider.h:67 still mentions "domains" in the PAM
>  response packet 
Fixed

>  * I'm not sure about removing pd->domain from
> pam_print_data(). It is
>  still used elsewhere in the code (via the DEBUG_PAM_DATA() macro) 
Ah, thanks. I removed that because I was originally going to remove
pd->domain from the pam_ctx struct, but I decided to leave it around. I
forgot to put it back into pam_print_data().

>  Otherwise looks good to me.
>   
New patch attached.


>  Ack to the other (DP registration) patch. 

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCv0AACgkQeiVVYja6o6OI9gCdES4LpP+zsVOJOPdLaD4qUaRe
> NbsAn3gvThQYCR4cGX/VUgKSsUN13OjZ
> =+J1z
> -----END PGP SIGNATURE-----

[Jakub Hrozek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/22/2010 06:30 PM, Stephen Gallagher wrote:
>  New patch attached.
>  
>   
Ack
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCwmUACgkQHsardTLnvCV2wQCfVJIgYwfgj0bscyoKhTZenF5d
> cTIAoLs0HDkxcvPHQvAnjbUH+cXiPLwD
> =mTkb
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/22/2010 12:44 PM, Jakub Hrozek wrote:
>  On 02/22/2010 06:30 PM, Stephen Gallagher wrote:
> > New patch attached.
>  
>  
>  
>  Ack 
Pushed both to master.
- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkuCxQkACgkQeiVVYja6o6NtVgCgoG0u1atm8lI8VeZVIhf7Wk1w
> u7YAn1TnevBkRzCo7d1/nwcb6sCrhSu3
> =8Ixc
> -----END PGP SIGNATURE-----