From: Tetsuo Handa <[EMAIL PROTECTED]>
Date: Mon, 09 Jul 2007 14:33:01 +0900

> @@ -649,8 +660,16 @@ int sock_recvmsg(struct socket *sock, st
>       init_sync_kiocb(&iocb, NULL);
>       iocb.private = &siocb;
>       ret = __sock_recvmsg(&iocb, sock, msg, size, flags);
> -     if (-EIOCBQUEUED == ret)
> +     if (-EIOCBQUEUED == ret) {
>               ret = wait_on_sync_kiocb(&iocb);
> +             /* I can now check security_socket_post_recvmsg(). */
> +             if (ret >= 0) {
> +                     int err = security_socket_post_recvmsg(sock, msg, size,
> +                                                            flags);
> +                     if (err)
> +                             ret = err;
> +             }
> +     }
>       return ret;
>  }

I don't think it's such a hot idea to return errors if the
wait_on_sync_kiocb() has returned success.

-
To unsubscribe from this list: send the line "unsubscribe 
linux-security-module" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to