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@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