- 15 Oct, 2010 34 commits
-
-
Haiyang Zhang authored
Convert camel cased local variable names in channel_mgmt.c to lower case Signed-off-by:
Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by:
Hank Janssen <hjanssen@microsoft.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Haiyang Zhang authored
Convert camel cased parameter in channel_interface.h to lower case Signed-off-by:
-
Haiyang Zhang authored
Convert camel cased function names in channel_interface.c to lower cases Signed-off-by:
-
Haiyang Zhang authored
Convert camel cased local variables in channel_interface.c to lower cases Signed-off-by:
-
Julia Lawall authored
In the first hunk, 0 is returned on memory allocation failure, even though other failures return -ENOMEM or other similar values. In the second hunk, there was error handling code that returned without freeing psz_path_name. A simplified version of the semantic match that finds the first problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression ret; expression x,e1,e2,e3; @@ ret = 0 ... when != ret = e1 *x = \(kmalloc\|kcalloc\|kzalloc\)(...) ... when != ret = e2 if (x == NULL) { ... when != ret = e3 return ret; } // </smpl> Signed-off-by:
Julia Lawall <julia@diku.dk> Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
Marek Belisko <marek.belisko@gmail.com> Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Marek Belisko authored
Signed-off-by:
-
Dan Carpenter authored
&sst_drv_ctx->streams[str_id].lock and &stream->lock are the same. The mutex_lock() uses &stream->lock so this makes things consistent and it's nicer to read as well. Signed-off-by:
Dan Carpenter <error27@gmail.com> Acked-by:
Vinod Koul <vinod.koul@intel.com> Signed-off-by:
-
Dan Carpenter authored
Using pdpram_blk here would cause an oops. We intended to use &freercvpool. Signed-off-by:
-
Dan Carpenter authored
This is a typo. It's supposed to be IS_ERR() here instead of PTR_ERR(). Signed-off-by:
Jarod Wilson <jarod@redhat.com> Signed-off-by:
-
Dan Carpenter authored
The code here is supposed checking if we exited the loop without hitting a break. The problem is that in the error handling "i" is out of bounds and it corrupts memory when we do an info->app_info[i].nRxMsgMiss++. >From the comments, it looks like someone noticed this corruption and updated the code, but didn't totally fix the problem. The correct fix is just to remove nRxMsgMiss++ from the error path. I believe this bug can be triggered remotely. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Replace a function-like macro with an inline function. * Remove an incorrect use of the volatile qualifier. The previously-volatile variable is now always protected by a spinlock. Signed-off-by:
Christopher Brannon <chris@the-brannons.com> Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Make the file_operations structure const. * Use strict_strtoul instead of simple_strtoul. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Remove an unused function-like macro. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Eliminate the unused synth_readable macro. * Convert function-like macros to inline functions. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Replace function-like macros with inline functions. * Simplify some boolean expressions. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Replace function-like macros with inline macros. * Eliminate a use of the ternary operator, for readability. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Eliminate an unused function-like macro. Signed-off-by:
-
Christopher Brannon authored
* Clean this file based on reports from checkpatch.pl. * Eliminate the unused synth_readable macro. * Convert function-like macros to inline functions. Signed-off-by:
-
- 14 Oct, 2010 6 commits
-
-
Pavan Savoy authored
Based on comments from Jiri Slaby, drop the register storage specifier, remove the unused code, cleanup the const to non-const type casting. Also make the line discipline ops structure static, since its a singleton, unmodified structure which need not be in heap. Reported-by:
Jiri Slaby <jirislaby@gmail.com> Signed-off-by:
Pavan Savoy <pavan_savoy@ti.com> Signed-off-by:
-
mike.rapoport@gmail.com authored
from 'struct osl_info' structure Signed-off-by:
Mike Rapoport <mike.rapoport@gmail.com> Signed-off-by:
-
mike.rapoport@gmail.com authored
Signed-off-by:
-
mike.rapoport@gmail.com authored
Signed-off-by:
-
mike.rapoport@gmail.com authored
Signed-off-by:
-
mike.rapoport@gmail.com authored
because there are other means to track memory leaks in kernel Signed-off-by:
-
