Send connman mailing list submissions to
[email protected]
To subscribe or unsubscribe via the World Wide Web, visit
https://lists.01.org/mailman/listinfo/connman
or, via email, send a message with subject or body 'help' to
[email protected]
You can reach the person managing the list at
[email protected]
When replying, please edit your Subject line so it is more specific
than "Re: Contents of connman digest..."
Today's Topics:
1. Re: [PATCH 2/2] service: Remove __connman_service_get_order()
(Daniel Wagner)
2. Re: [PATCH] plugins: ethernet: get rid of truncate warning
(Marcus Folkesson)
----------------------------------------------------------------------
Message: 1
Date: Wed, 29 Aug 2018 20:54:41 +0200
From: Daniel Wagner <[email protected]>
To: Slava Monich <[email protected]>
Cc: [email protected]
Subject: Re: [PATCH 2/2] service: Remove __connman_service_get_order()
Message-ID: <[email protected]>
Content-Type: text/plain; charset=utf-8; format=flowed
Hi Slava,
Both patches applied.
On 08/29/2018 04:43 PM, Slava Monich wrote:
> It doesn't do anything that's not done by set_split_routing()
> and its return value was never used.
No idea what Patrik wanted to do with that function. Anyway, it was
doing nothing for a long time so no need to keep it around.
Thanks!
Daniel
------------------------------
Message: 2
Date: Wed, 29 Aug 2018 20:56:21 +0200
From: Marcus Folkesson <[email protected]>
To: Daniel Wagner <[email protected]>
Cc: [email protected]
Subject: Re: [PATCH] plugins: ethernet: get rid of truncate warning
Message-ID: <[email protected]>
Content-Type: text/plain; charset="utf-8"
Hi Daniel,
On Wed, Aug 29, 2018 at 08:37:43PM +0200, Daniel Wagner wrote:
> Hi Marcus,
>
> On 08/27/2018 02:32 PM, Marcus Folkesson wrote:
> > The buffer will never be truncated in reality, but it is the only
> > warning during compile... so get rid of it.
>
> I suppose this is a very new gcc?
>
Not the most recent, but still pretty new:
gcc (GCC) 8.1.1 20180531
> >
> > The ethtool_cmd.phy_address used as dsaport is an __u8, so a `char` is
> > sufficient.
> >
> > Use `char` as dsaport to not make snprintf nervous about truncating
> > something.
> >
> > Warning:
> > ----------------------->8-----------------------------------
> >
> > plugins/ethernet.c: In function ?ethernet_newlink?:
> > plugins/ethernet.c:208:52: warning: ?__builtin___snprintf_chk? output may
> > be truncated before the last format character [-Wformat-truncation=]
> > snprintf(group, sizeof(group), "p%02x_%03x_cable", dsaport, vid);
> > ^
> > In file included from /usr/include/stdio.h:862,
> > from plugins/ethernet.c:31:
> > /usr/include/bits/stdio2.h:64:10: note: ?__builtin___snprintf_chk? output
> > between 14 and 17 bytes into a destination of size 16
> > return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1,
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > __bos (__s), __fmt, __va_arg_pack ());
> > ----------------------->8-----------------------------------
> >
> > Signed-off-by: Marcus Folkesson <[email protected]>
>
> Patch applied. I dropped the SoB because we don't do it here.
Thanks, saw you mail about v2 :-)
>
> Thanks,
> Daniel
Best regards,
Marcus Folkesson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL:
<http://lists.01.org/pipermail/connman/attachments/20180829/dfaab84e/attachment-0001.asc>
------------------------------
Subject: Digest Footer
_______________________________________________
connman mailing list
[email protected]
https://lists.01.org/mailman/listinfo/connman
------------------------------
End of connman Digest, Vol 34, Issue 20
***************************************
