Hello,
I am extremely sorry for the inconvenience. The patch is attached with the
mail. Kindly have a look. I will shortly send another patch for authtor
module with the suggested fix.
Thanking You,
Pallavi
On 11 November 2013 17:05, Lukas Slebodnik <lslebodn(a)redhat.com> wrote:
On (09/11/13 19:29), Pallavi Jha wrote:
>Hi
>
>I have created the patch and attached it with this mail. Kindly review it.
>I have commented some of the test as it fails(gives segmentation fault)
>when authtoken is NULL. I think we should test tok for NULL before
>accessing its elements. Please correct me if am wrong.
>
>for example :
>
>uint8_t *sss_authtok_get_data(struct sss_auth_token *tok)
>{
> return tok->data;
>}
>
>This should be written as:
>
>uint8_t *sss_authtok_get_data(struct sss_auth_token *tok)
>{
> if(!tok) {
> return EINVAL;
> }
> return tok->data;
>}
>
>Thanking You,
>Pallavi
I cannot find any attachment.
LS