diff options
| author | Grant Likely <grant.likely@secretlab.ca> | 2008-03-22 06:41:05 +0300 |
|---|---|---|
| committer | Paul Mackerras <paulus@samba.org> | 2008-03-24 09:55:49 +0300 |
| commit | 9560aea4e9d17cb75113c6051e800222fd5c71a4 (patch) | |
| tree | 88517a2a9d2380196b32b03f047eadf1602316c8 /arch/powerpc/sysdev/bestcomm | |
| parent | b8c19eb16a7e6df57d0f6d67e42ce026e5d5930b (diff) | |
| download | linux-9560aea4e9d17cb75113c6051e800222fd5c71a4.tar.xz | |
[POWERPC] mpc5200: Fix null dereference if bestcomm fails to initialize
If the bestcomm initialization fails, calls to the task allocate
function should fail gracefully instead of oopsing with a NULL deref.
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/sysdev/bestcomm')
| -rw-r--r-- | arch/powerpc/sysdev/bestcomm/bestcomm.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/powerpc/sysdev/bestcomm/bestcomm.c b/arch/powerpc/sysdev/bestcomm/bestcomm.c index f589999361e0..b18cab55a76d 100644 --- a/arch/powerpc/sysdev/bestcomm/bestcomm.c +++ b/arch/powerpc/sysdev/bestcomm/bestcomm.c @@ -52,6 +52,10 @@ bcom_task_alloc(int bd_count, int bd_size, int priv_size) int i, tasknum = -1; struct bcom_task *tsk; + /* Don't try to do anything if bestcomm init failed */ + if (!bcom_eng) + return NULL; + /* Get and reserve a task num */ spin_lock(&bcom_eng->lock); |