Re: [PATCH v2] mac80211: add debug file for mic failure

[Johannes Berg <johannes@sipsolutions.net>]

On Wed, 2012-12-05 at 12:24 +0530, Saravana wrote:
> On 12/05/2012 12:37 AM, Johannes Berg wrote:
> > On Tue, 2012-12-04 at 15:57 +0100, Johannes Berg wrote:
> >> On Tue, 2012-12-04 at 19:47 +0530, Saravana wrote:
> >>> The mic failure count will provide the number of mic failure that 
> >>> have happened without a countermeasure being started. Once the 
> >>> countermeasure is started, the counter will be reset to 0.
> >>> This count will be helpful when the consecutive 
> >>> mic failure occur at intervals greater than 60 seconds regularly(frequently).
> >>
> >> Applied.
> > 
> > Never mind, dropped. It has a null pointer issue and still has
> 
> could the null pointer issue be prevented by checking 
> the rx->key is valid before update the mic failure? Is it acceptable?
> 
> 	if (rx->key)
> 		rx->key->u.tkip.mic_failures++

Should probably use two labels instead.

> > whitespace issues. Please figure out how to actually indent
> > parameters ...
> > 
> The patch was verified with the checkpatch script. it didn't point out whitespace
> error and the indentation issue.Is there any other scripts that needs to be run additionally?
> Please let me know how to figure out this issue ?

*shrug*, not sure. You've indented the function arguments wrong:

static ssize_t key_mic_failures_read(struct file ...
					size_t count)

you should use spaces there at the end to line up right under "struct",
not only tabs that go too far (or not far enough)

> > Also, signed-off-by should be your full name, is that really just
> > "Saravana"? Your email seems to indicate there's something else
> > ("saravanad")
> > 
> 
> My full name is Saravanakumar and my Father name is Doraiswami. 
> Saravanad, d-indicates my starting letter of my Father name.
> I will probably use Saravanakumar in the future patches..

Hah, sorry. I make this mistake all the time, my bad, I apologise. No
need to do anything differently.

Anyway ... I've fixed it up for you now.

johannes