Re: [chrony-dev] [GIT] chrony/chrony.git branch, master, updated. 2.2.1-105-ge3191e3

[Bryan Christianson <bryan@xxxxxxxxxxxxx>]

More compiler warnings in OSX


gcc -O2 -g -Wmissing-prototypes -Wall -pthread   -c client.c
client.c:2128:39: warning: format specifies type 'unsigned short' but the
      argument has type 'unsigned int' [-Wformat]
             ntohl(client->ntp_hits), ntohl(client->ntp_drops));
                                      ^~~~~~~~~~~~~~~~~~~~~~~~
/SDKs/MacOSX10.11.sdk/usr/include/sys/_endian.h:135:18: note: expanded from
      macro 'ntohl'
#define ntohl(x)        __DARWIN_OSSwapInt32(x)
                        ^~~~~~~~~~~~~~~~~~~~~~~
/SDKs/MacOSX10.11.sdk/usr/include/libkern/_OSByteOrder.h:75:5: note: expanded
      from macro '__DARWIN_OSSwapInt32'
    (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt32(x) : _OSSwapInt32(x))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
client.c:2135:39: warning: format specifies type 'unsigned short' but the
      argument has type 'unsigned int' [-Wformat]
             ntohl(client->cmd_hits), ntohl(client->cmd_drops));
                                      ^~~~~~~~~~~~~~~~~~~~~~~~
/SDKs/MacOSX10.11.sdk/usr/include/sys/_endian.h:135:18: note: expanded from
      macro 'ntohl'
#define ntohl(x)        __DARWIN_OSSwapInt32(x)
                        ^~~~~~~~~~~~~~~~~~~~~~~
/SDKs/MacOSX10.11.sdk/usr/include/libkern/_OSByteOrder.h:75:5: note: expanded
      from macro '__DARWIN_OSSwapInt32'
    (__builtin_constant_p(x) ? __DARWIN_OSSwapConstInt32(x) : _OSSwapInt32(x))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.

> On 30/01/2016, at 5:57 AM, git@xxxxxxxxxxxxx wrote:
> 
> This is an automated email from git. It was enerated because a ref
> change was pushed to the repository "chrony/chrony.git".
> 
> The branch, master has been updated
>       via  e3191e372b9925333f66bf36908f504e1dde7c12 (commit)
>       via  705e32acdc2e438e7d9a7394560d0f2d63054129 (commit)
>       via  6e4dd9302df5e6d576a18e22bd86167d0296a7d2 (commit)
>       via  ea002130d720ea0d365f216167d74ac53f97b237 (commit)
>       via  7ba5ffa706821da789130185cdbfd3013e227cde (commit)
>       via  861ac013bcc54fa7b3c92cc4b43a8e2071885907 (commit)
>       via  a6da963f45011c8f13fc6a2991101808d90e9634 (commit)
>       via  55ba7ee2a180cb2b82da165b8b083da170eeb569 (commit)
>       via  3121f31cedaa682561e22db6efdf2f9627929ca4 (commit)
>       via  da296db91ded60c583099be0319623e37d24a85f (commit)
>       via  d36ca9288a77c519563299ab594862de7886fae1 (commit)
>       via  8549043a3fb0e9ea659d888aec2fe4c8bb4e5ea6 (commit)
>       via  e0ae2b4bb52240aa8b931e96e7f8da6571caa11a (commit)
>       via  aad42ceaec7d0685ca01e93fc19df555302aebda (commit)
>      from  f225469e6ed93be779c3e4ae9edbc41c73fa22b5 (commit)
> 
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
> 
> - Log -----------------------------------------------------------------
> commit e3191e372b9925333f66bf36908f504e1dde7c12
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 16:50:43 2016 +0100
> 
>    cmdmon: update protocol changelog
> 
> commit 705e32acdc2e438e7d9a7394560d0f2d63054129
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 15:46:56 2016 +0100
> 
>    cmdmon: define new types for CLIENT_ACCESSES_BY_INDEX command
> 
>    There was an incompatible change in the client access report. To avoid
>    bumping the protocol version drop support for the original request/reply
>    types and define new CLIENT_ACCESSES_BY_INDEX2 types as a newer version
>    of the command.
> 
> commit 6e4dd9302df5e6d576a18e22bd86167d0296a7d2
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 15:35:32 2016 +0100
> 
>    cmdmon: allow unhandled commands
> 
>    Replace the assert() with a debug message to not crash if someone
>    forgets to implement a newly defined command.
> 
> commit ea002130d720ea0d365f216167d74ac53f97b237
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 15:32:47 2016 +0100
> 
>    cmdmon: reply to invalid commands
> 
>    If an unknown command is received (e.g. from a future client), it should
>    get a reply and print an error code instead of timing out.
> 
> commit 7ba5ffa706821da789130185cdbfd3013e227cde
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 15:06:43 2016 +0100
> 
>    cmdmon: update debug messages
> 
> commit 861ac013bcc54fa7b3c92cc4b43a8e2071885907
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 13:46:38 2016 +0100
> 
>    cmdmon: use 32-bit fields in client access report
> 
>    The clientlog record still uses 16-bit integers to count dropped
>    packets, but this will avoid an incompatible change in the command
>    reply if there will be a need to count more than 2^16 drops.
> 
> commit a6da963f45011c8f13fc6a2991101808d90e9634
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Thu Jan 28 14:52:34 2016 +0100
> 
>    clientlog: don't allow rate limiting with noclientlog
> 
> commit 55ba7ee2a180cb2b82da165b8b083da170eeb569
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Fri Jan 29 17:18:14 2016 +0100
> 
>    doc: update description of clients command
> 
> commit 3121f31cedaa682561e22db6efdf2f9627929ca4
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Thu Jan 28 14:23:26 2016 +0100
> 
>    doc: describe rate limiting directives
> 
> commit da296db91ded60c583099be0319623e37d24a85f
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Mon Jan 25 17:55:20 2016 +0100
> 
>    examples: update for recent changes
> 
> commit d36ca9288a77c519563299ab594862de7886fae1
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Mon Jan 25 17:24:08 2016 +0100
> 
>    doc: update keyfile description
> 
> commit 8549043a3fb0e9ea659d888aec2fe4c8bb4e5ea6
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Tue Jan 26 15:35:55 2016 +0100
> 
>    conf: set logchange to 1 second by default
> 
>    logchange is now always enabled, with 1 second threshold by default.
> 
> commit e0ae2b4bb52240aa8b931e96e7f8da6571caa11a
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Mon Jan 25 18:03:39 2016 +0100
> 
>    client: generate key 1 by default in keygen command
> 
> commit aad42ceaec7d0685ca01e93fc19df555302aebda
> Author: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
> Date:   Mon Jan 25 16:50:51 2016 +0100
> 
>    keys: warn about short key only if used by source
> 
>    After restricting authentication of servers and peers to the specified
>    key, a short key in the key file is a security problem from the client's
>    point of view only if it's specified for a source.
> 
> -----------------------------------------------------------------------
> 
> Summary of changes:
> candm.h                       |   20 +++--
> chrony.texi.in                |  172 +++++++++++++++++++++++++++--------------
> client.c                      |   17 ++--
> clientlog.c                   |   33 ++++----
> cmdmon.c                      |   40 +++++-----
> conf.c                        |   15 ++--
> conf.h                        |    2 +-
> examples/chrony.conf.example2 |    6 --
> examples/chrony.conf.example3 |   29 ++++---
> examples/chrony.keys.example  |   15 ++--
> keys.c                        |   18 ++++-
> keys.h                        |    1 +
> ntp_core.c                    |    9 ++-
> pktlength.c                   |    8 +-
> reference.c                   |   13 +---
> 15 files changed, 228 insertions(+), 170 deletions(-)
> 
> 
> hooks/post-receive
> --
> chrony/chrony.git
> 
> -- 
> To unsubscribe email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "unsubscribe" in the subject.
> For help email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "help" in the subject.
> Trouble?  Email listmaster@xxxxxxxxxxxxxxxxxxxx.
> 



Bryan Christianson
bryan@xxxxxxxxxxxxx




--
To unsubscribe email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "unsubscribe" in the subject.
For help email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "help" in the subject.
Trouble?  Email listmaster@xxxxxxxxxxxxxxxxxxxx.