diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-10-15 22:36:48 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-15 15:42:26 -0700 |
| commit | 9f407840bb888e683fa844b1b80eec54ced84653 (patch) | |
| tree | 83acf0ce385a9df5733d5a1fd5b7cbdbc298233a /drivers/staging/intel_sst/intel_sst_fw_ipc.h | |
| parent | 87b554a04a0d12f8ff3612b6c2e163e29a34616a (diff) | |
| download | linux-9f407840bb888e683fa844b1b80eec54ced84653.tar.bz2 | |
Staging: intel_sst: off by one bug
This should be >= instead of > or we go passed the end of the array.
Also the arrays are declared with size MAX_NUM_STREAMS. This is the
only place that uses MAX_NUM_STREAMS_MFLD. It seems like asking for
trouble to use two variables for the same information. I've changed
everything to use MAX_NUM_STREAMS.
This bug isn't really harmful. In the worst case, if you enabled
debugging then you would see a message.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/intel_sst/intel_sst_fw_ipc.h')
| -rw-r--r-- | drivers/staging/intel_sst/intel_sst_fw_ipc.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/intel_sst/intel_sst_fw_ipc.h b/drivers/staging/intel_sst/intel_sst_fw_ipc.h index 1a2f67f0aedc..9d3c36807e07 100644 --- a/drivers/staging/intel_sst/intel_sst_fw_ipc.h +++ b/drivers/staging/intel_sst/intel_sst_fw_ipc.h @@ -31,7 +31,6 @@ */ #define MAX_NUM_STREAMS_MRST 3 -#define MAX_NUM_STREAMS_MFLD 6 #define MAX_NUM_STREAMS 6 #define MAX_DBG_RW_BYTES 80 #define MAX_NUM_SCATTER_BUFFERS 8 |