Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

why MaxRepetitions is uint8 #77

Closed
pingliu opened this issue Aug 31, 2016 · 2 comments
Closed

why MaxRepetitions is uint8 #77

pingliu opened this issue Aug 31, 2016 · 2 comments

Comments

@pingliu
Copy link

pingliu commented Aug 31, 2016

when I use bulkget, the MaxRepetitions value is bigger than 255,because the cpu num is bigger than 255.

so,why MaxRepetitions is uint8,when use int32,what will happen

@soniah
Copy link
Collaborator

soniah commented Aug 31, 2016

Thanks, someone will look at this.

On Wednesday, 31 August 2016, pingliu notifications@github.com wrote:

when I use bulkget, the MaxRepetitions value is bigger than 255,because
the cpu num is bigger than 255.

so,why MaxRepetitions is uint8,when use int32,what will happen


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#77, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbJm88odD-TsGN3wCdrGCk6TTh86leks5qlSxvgaJpZM4JxTyl
.

Sent from my phone.

soniah added a commit that referenced this issue Sep 1, 2016
@soniah
Copy link
Collaborator

soniah commented Sep 1, 2016

Updated with the following comment:

// GetBulk sends an SNMP GETBULK request                                                                             
//                                                                                                                   
// For maxRepetitions greater than 255, use BulkWalk() or BulkWalkAll()
func (x *GoSNMP) GetBulk(oids []string, nonRepeaters uint8, maxRepetitions uint8) (result *SnmpPacket, err error) { 

In addition, MaxRepetitions was changed to uint8

@soniah soniah closed this as completed Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants