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.

Reply via email to