diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2017-11-08 11:21:52 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-11-08 11:21:52 -0800 |
| commit | d6a2cf07f0c91e339d2c53f1e1ca6e731af2e72a (patch) | |
| tree | 191177732b23ff3e4dedac329372349bb6ba46f5 | |
| parent | f7dc4c9a855a13dbb33294c9fc94f17af03f6291 (diff) | |
| parent | 624f5ab8720b3371367327a822c267699c1823b8 (diff) | |
| download | linux-samsung-d6a2cf07f0c91e339d2c53f1e1ca6e731af2e72a.tar.gz | |
Merge branch 'fixes-v4.14-rc8' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security
Pull key handling fix from James Morris:
"Fix by Eric Biggers for the keys subsystem"
* 'fixes-v4.14-rc8' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security:
KEYS: fix NULL pointer dereference during ASN.1 parsing [ver #2]
| -rw-r--r-- | lib/asn1_decoder.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/asn1_decoder.c b/lib/asn1_decoder.c index fef5d2e114be1e..1ef0cec38d7879 100644 --- a/lib/asn1_decoder.c +++ b/lib/asn1_decoder.c @@ -228,7 +228,7 @@ next_op: hdr = 2; /* Extract a tag from the data */ - if (unlikely(dp >= datalen - 1)) + if (unlikely(datalen - dp < 2)) goto data_overrun_error; tag = data[dp++]; if (unlikely((tag & 0x1f) == ASN1_LONG_TAG)) @@ -274,7 +274,7 @@ next_op: int n = len - 0x80; if (unlikely(n > 2)) goto length_too_long; - if (unlikely(dp >= datalen - n)) + if (unlikely(n > datalen - dp)) goto data_overrun_error; hdr += n; for (len = 0; n > 0; n--) { |