diff options
| author | philip <philip@FreeBSD.org> | 2008-11-17 19:00:36 +0000 |
|---|---|---|
| committer | philip <philip@FreeBSD.org> | 2008-11-17 19:00:36 +0000 |
| commit | f3d74d62d37e8bf8ef5dd002e75273cfa6951166 (patch) | |
| tree | b8a181a4c28e050f744e58848c41f6efe6cefa01 /sys/crypto | |
| parent | ee3d6ade2edc21fcfce1a1bb9dc49df4b0cf4367 (diff) | |
| download | FreeBSD-src-f3d74d62d37e8bf8ef5dd002e75273cfa6951166.zip FreeBSD-src-f3d74d62d37e8bf8ef5dd002e75273cfa6951166.tar.gz | |
Fix a potential NULL-pointer dereference in padlock(4).
Spotted by: Coverity (via pjd)
MFC after: 1 week
Diffstat (limited to 'sys/crypto')
| -rw-r--r-- | sys/crypto/via/padlock.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/crypto/via/padlock.c b/sys/crypto/via/padlock.c index 4157c47..dc97a88 100644 --- a/sys/crypto/via/padlock.c +++ b/sys/crypto/via/padlock.c @@ -297,7 +297,11 @@ padlock_process(device_t dev, struct cryptop *crp, int hint __unused) enccrd = maccrd = NULL; - if (crp == NULL || crp->crp_callback == NULL || crp->crp_desc == NULL) { + /* Sanity check. */ + if (crp == NULL) + return (EINVAL); + + if (crp->crp_callback == NULL || crp->crp_desc == NULL) { error = EINVAL; goto out; } |
