snmp.Get(m.snmpOids[i:end]) randomly crashes about gosnmp HOT 10 CLOSED

I've uploaded the generated debug file to this gist, if you need some more info.
https://gist.github.com/toni-moreno/40dc6353127fbd21828afc6108699100

from gosnmp.

According to that dump, the SNMP agent is responding with error TooBig with
nothing supplied in the VBL.

@codedance

https://github.com/soniah/gosnmp/blame/master/marshal.go#L282

Are we sure an empty VBL is an error of this magnitude?

On Mon, Apr 11, 2016 at 8:08 AM, Toni Moreno [email protected]
wrote:

I've uploaded the generated debug file to this gist, if you need some more
info.
https://gist.github.com/toni-moreno/40dc6353127fbd21828afc6108699100

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#62 (comment)

Whitham D. Reeve II
907-764-5263

from gosnmp.

Hi everyone, I'm having laser surgery on my eyes tomorrow - better vision :-) But that means I'll be taking a break for two weeks from programming :-/

from gosnmp.

Hi @soniah how are you ? I'm done a workaround by changing snmpget to snmpgetbulk method to get indexed OID's, but only working with snmp version > 2 , whould like to fix this error for old snmp version 1 devices.

Thanks a lot!

from gosnmp.

Soniah, what do you want to do about this? I often have to use SNMPV1 for ... reasons. I am likely to be affected by this at some point.

We could try removing the check to allow code higher up the opportunity to deal with it. Need to verify by looking through the call graph. Does that seem like a good idea to you?

from gosnmp.

Hi @wdreeveii , I've had a look through the code and don't understand the implications - but I'm at a campsite with mosquitos and a flat laptop :-)

I've been trying to understand the effect of removing this:

(line 282)

len(result.Variables) < 1

ie if unmarshal returns a SnmpPacket with no Variables, why it is a problem.

A call graph on v1 protocol would be handy, I can't remember the protocol differences between v1 & v2c

from gosnmp.

This relates to the issue I opened #72

My issue was more around allowing the specification of the maximum OIDs requested but the error still goes unhandled from what I can see in the code.

Is there anything ongoing currently to support SNMP error codes returned from the remote agent?

from gosnmp.

I'm not sure. At the moment I'm focussing on the trap code.

from gosnmp.

Notes for myself:

• Max oids exceeded is now handled (eg gosnmp.go, line 310).
• returned SNMP error codes....?

from gosnmp.

Closing as stale. @toni-moreno please log a new issue if this is still relevant.

from gosnmp.