2015-03-24 14:37 GMT+01:00 Andrew Ruch <[email protected]>: > 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?
It should match the variable name (just FYI). I did a quick check, pszStrucData looks right. Thx! Rainer _______________________________________________ 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.
