DefaultPDUFactory contextEngineID/contextName initialization inconsistency

SNMP4J
1

Hello!

DefaultPDUFactory no-args constructor leaves contextEngineID and contextName fields uninitialized. This state can trigger NPE when createPDU method tries to apply context to just instantiated scoped PDU.
Constructor with single PDU type argument initializes these fields to empty strings.
To ensure consistent behaviour, default constructor should do the same.

2

Hello Boris,

Valid point, I have fixed the default constructor to

DefaultPDUFactory() { this(PDU.GET); }

The ticket ID for this fix is SFJ-206.

Best regards
Frank