Add Management frame to ieee80211 module about dpkt HOT 11 CLOSED

I attached a patch that supports management frames, as well as few other types. 
It also has test cases to check the code.

 Dear timur alperovich,
 I add nformation elements to your patch. we could talk about the field names that I add. If you want?

Regards,
Sylvain  

Hi Sylvain,

I'm a little bit confused because it looks like the latest patch you added was 
a diff against dpkt with your previous patch applied?

I agree that it makes sense to have specific IE names, etc, because right now, 
as you noticed, I just unpack all of them and don't do anything further with 
them (aside from SSID and DS IEs). It would probably make sense to unpack each 
IE with length and ID and use the ID to lookup the appropriate class to unpack 
the rest of it.

I'm guessing that's what you're suggesting to add?

Cheers,
Timur.

Hi Timur,

I'm sorry, i attach the wrong file, i made a new patch with your diff,

give me your opinion on it ;)

Regards,
Sylvain 

Hi Sylvain,

Looks great. I think that's the right way to do this. Would you mind uploading 
a joint patch next, because none of these have been applied to dpkt and it 
would make sense to combine the changes?

One other thing, could you please add tests to make sure the IE decoding works 
correctly? You can probably craft a very simple packet or even add to the 
beacon packet I have in the prior tests. The tests are important for regression 
testing if there is more code added to the ieee80211 parser (and I'm sure there 
will be).

Thank you,
Timur

Hi Timur/Sylvain, 

Is the patch applied in latest revision? Where can I get the complete updated 
files with changes you guys have made?

Thanks,
Arun

[deleted comment]

Indeed - we'll get this in. Sorry for the delay, guys. There's a missing llc.py 
referenced in the tests; can you post that as well?

this is the last patch i have done with the lastest revision

I have committed llc.py and the changes to ieee80211 that I and Sylvain have 
made.

Arun: the patch is now applied and you can go ahead and use it. Post back if 
you run into problems with it. I fixed one bug, but there could be more :)