[ECOS] Limitations with the SNTP client? - patch

[Gary Thomas]

Andrew Lunn wrote:
> On Wed, Nov 08, 2006 at 09:57:34AM -0500, Andre-John Mas wrote:
>> Do I post this here? I have never posted a patch before, so I am kinda
>> new to this.
> 
> The ecos-patches mailling list please.
> 
>> Removed optional KeyIdentifer and MesageDigest fields, which 
>> weren't being used, and caused certain servers not to respond
>> when these fields were included with null values.
> 
> Please put this in the ChangeLog file in the sntp directory. Follow
> the format of the other entries.
>  
>> ecos-opt/net/net/sntp/current/src/sntp.c
>>
>> 75a76,80
>>> /* Note that the KeyIdentifier and MessageDigest fields are not
>>>    included since they are optional (RFC2030) and in certain cases
>>>    will cause some SNTP servers from responding if included with
>>>    null values.
>>>  */
>> 93,94d97
>> <   cyg_uint32    KeyIdentifier;          /* Optional */
>> <   cyg_uint8     MessageDigest[16];      /* Optional */
> 
> Please use the unified format to diff, ie -u. It is easier to read.
>

I would prefer to see these just commented out (or even a CDL option).
That way, we can remember what goes there if it's ever necessary to
resurrect those fields.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss