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

IGMP checksum fails #20

Closed
GoogleCodeExporter opened this issue Mar 27, 2015 · 3 comments
Closed

IGMP checksum fails #20

GoogleCodeExporter opened this issue Mar 27, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Create a packet using ImpactPacket.IGMP()
2. Send the packet

What is the expected output? What do you see instead?
Expect an IGMP packet to be constructed, but get:

  File "/Library/Python/2.7/site-packages/impacket/ImpactPacket.py", line 1864, in calculate_checksum
    if self.__auto_checksum and (not self.get_igmp_cksum()):
AttributeError: IGMP instance has no attribute '_IGMP__auto_checksum'

What version of the product are you using? On what operating system?
svn version on OSX 10.8

Please provide any additional information below.

The patch fixing the issue is attached

Original issue reported on code.google.com by diogogo...@gmail.com on 20 Oct 2012 at 12:01

Attachments:

@GoogleCodeExporter
Copy link
Author

Hey...

Thanks for the patch!.. Just fixed it in 
http://code.google.com/p/impacket/source/detail?r=728

thanks again,
bto

Original comment by bet...@gmail.com on 22 Oct 2012 at 4:55

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Welcome!

Glad I could help, do you take patches with new methods (I'm thinking here of 
higher level methods to tinker with IGMP)

Original comment by diogogo...@gmail.com on 22 Oct 2012 at 5:02

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Hey mate...

Absolutely.. assuming they don't interfere with other existing methods.. feel 
free to send them along.. 

Thanks again! :)

bto

Original comment by bet...@gmail.com on 22 Oct 2012 at 5:18

  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant