On Tue, Mar 24, 2015 at 7:26 AM, Rainer Gerhards
<[email protected]> wrote:
> 2015-03-24 14:18 GMT+01:00 Andrew Ruch <[email protected]>:
>> On Tue, Mar 10, 2015 at 4:18 PM, Andrew Ruch <[email protected]> wrote:
>>> On Fri, Mar 6, 2015 at 8:53 AM, Andrew Ruch <[email protected]> wrote:
>>>> On Thu, Mar 5, 2015 at 11:45 AM, Andrew Ruch <[email protected]> wrote:
>>>>> On Thu, Mar 5, 2015 at 11:38 AM, David Lang <[email protected]> wrote:
>>>>>> What do you mean when you say it's getting corrupted?
>> <SNIP>
>>>
>>>
>>> The MsgSerialize and MsgDeserialize functions are out of sync in
>>> msg.c. They label the structured data field differently and expect it
>>> in a different order. When I fix the name and fix the ordering in
>>> MsgDeserialize, my systems work fine.
>>>
>>> I created a bug (#264).
>>
>>
>> Any chance of this issue (#264) being fixed for the next release? I'd
>> be willing to submit a pull request, but I think it's only moving a
>> few lines. And I would need to know which function (MsgSerialize or
>> MsgDeserialize) is correct and which one should change.
>
> Please submit a pull request. If you place *your patch* under ASL 2.0,
> we can probably merge it relatively quickly. The order of properties
> does (should;)) not matter -- MsgDeserialize should just check the
> name in an if-elseif-else chain (if not, that is definitely wrong).
> But the wrong name of course causes troubles.
>
> Does that help?
> Rainer
>>
>> Thanks,
>> Andrew
>> _______________________________________________
>> rsyslog mailing list
>> http://lists.adiscon.net/mailman/listinfo/rsyslog
>> http://www.rsyslog.com/professional-services/
>> What's up with rsyslog? Follow https://twitter.com/rgerhards
>> NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of 
>> sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T 
>> LIKE THAT.
> _______________________________________________
> rsyslog mailing list
> http://lists.adiscon.net/mailman/listinfo/rsyslog
> http://www.rsyslog.com/professional-services/
> What's up with rsyslog? Follow https://twitter.com/rgerhards
> NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of 
> sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T 
> LIKE THAT.

Would you like the value called "pszRcvStrucData" from MsgSerialize or
"pszStrucData" from MsgDeserialize?
_______________________________________________
rsyslog mailing list
http://lists.adiscon.net/mailman/listinfo/rsyslog
http://www.rsyslog.com/professional-services/
What's up with rsyslog? Follow https://twitter.com/rgerhards
NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of 
sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE 
THAT.

Reply via email to