<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-02-26 1:03 GMT+01:00 Charles Lepple <span dir="ltr"><<a href="mailto:clepple@gmail.com" target="_blank">clepple@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><div class=""><div>On Feb 25, 2014, at 4:23 PM, Arnaud Quette wrote:</div>
<br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">gen-snmp-subdriver.sh simply hasn't done its job, for some reasons.</div></div><div class="gmail_extra">well, 1 clear is that's it not a completed effort... at all<br>

</div><div class="gmail_extra">but, still IIRC, it should have produced at least a bit more<br></div></div></blockquote></div><div dir="ltr"><div class="gmail_extra"><br></div><div class="gmail_extra">Hi Arnaud,</div></div>
</div></div></blockquote><div><br></div><div>Hi Charles<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word">
<div><div dir="ltr"><div class="gmail_extra"></div><div class="gmail_extra">I may not have been specific enough in my description.</div></div></div></div></blockquote><div> <br></div><div>I completely overlooked that point while focusing on the new subdrv consideration. sorry...<br>
</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><div dir="ltr"><div class="gmail_extra">
</div><div class="gmail_extra">The crash that Jason saw is with the version of snmp-ups that is currently in Git - including the new 'xppc' MIB, but the NULL pointer in question is also in 'delta_ups'.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Given that mib2nut[i]->oid_auto_check is only used for the "classic" OID check, I think we can just add a special case to check for NULL, and continue the loop if oid_auto_check has not been provided.</div>
</div></div></div></blockquote><div><br></div><div>you're fully right indeed!<br></div><div>but the root cause (and solution) is probably deeper:<br><a href="https://github.com/networkupstools/nut/commit/8428fa57614ffe777191d9ae127b53d964bca6f3">https://github.com/networkupstools/nut/commit/8428fa57614ffe777191d9ae127b53d964bca6f3</a><br>
</div><br><div>however, there is still some doc completion needed for the subdriver generation.<br></div>and lot more on this still-too-experimental code :-/<br><br>all in all, I'll have to complete quickly #20 specs:<br>
<a href="https://github.com/networkupstools/nut/issues/20">https://github.com/networkupstools/nut/issues/20</a> <br><div><br></div><br>@Jason: could you please try with the latest git master?<br><br></div><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><div class=""><blockquote type="cite"><div dir="ltr"><div class="gmail_extra">
@Jason: how have you called the script?<br clear="all"></div>
<div class="gmail_extra">could you please also send an archive including:<br></div><div class="gmail_extra">- the results of the adapted commands on #L73, #L74<br><a href="https://github.com/networkupstools/nut/blob/master/scripts/subdriver/gen-snmp-subdriver.sh" target="_blank">https://github.com/networkupstools/nut/blob/master/scripts/subdriver/gen-snmp-subdriver.sh</a><br>

</div><div class="gmail_extra">- your mib file<br></div></div></blockquote></div></div><div><br></div>Jason,<div><br></div><div>this would still be useful to address the issue of gen-snmp-subdriver not parsing things correctly.</div>
</div></blockquote></div><br></div><div class="gmail_extra">to be continued...<br></div><div class="gmail_extra"><br clear="all">Arno</div></div>