[oe] [PATCH] fix acpid compiler warning on some platforms

Paul Menzel paulepanter at users.sourceforge.net
Sun May 15 10:41:40 UTC 2011


Dear Christian,


Am Sonntag, den 15.05.2011, 00:16 -0400 schrieb Christian Betz:

thank you for your fast response. Next time please also add the commit
message. The one you provided in your first message would have been
enough. You can even change/improve commit messages using `git commit
--amend` (or `git rebase …`) before doing `git format-patch …`.

I will add the commit message when applying your patch.

You also forgot your Signed-off-by line. Please reply to this message
with just that line since it is required by our commit policy.

Passing `-s` to `git commit` or `git format-patch` adds that line
automatically to  the commit message.

> ---
>  recipes/acpid/acpid-1.0.10/netlink.diff |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/acpid/acpid-1.0.10/netlink.diff b/recipes/acpid/acpid-1.0.10/netlink.diff
> index 635c550..ebdec58 100644
> --- a/recipes/acpid/acpid-1.0.10/netlink.diff
> +++ b/recipes/acpid/acpid-1.0.10/netlink.diff
> @@ -2019,7 +2019,7 @@ diff -ruN acpid-1.0.10/libnetlink.c acpid-1.0.10-netlink2/libnetlink.c
>  +
>  +int addraw_l(struct nlmsghdr *n, int maxlen, const void *data, int len)
>  +{
> -+	if ((int)NLMSG_ALIGN(n->nlmsg_len) + NLMSG_ALIGN(len) > maxlen) {
> ++	if ((int)NLMSG_ALIGN(n->nlmsg_len) + (int)NLMSG_ALIGN(len) > maxlen) {
>  +		fprintf(stderr, "addraw_l ERROR: message exceeded bound of %d\n",maxlen);
>  +		return -1;
>  +	}


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110515/883066b5/attachment-0002.sig>


More information about the Openembedded-devel mailing list