diff options
| author | Eric Biggers <ebiggers3@gmail.com> | 2017-12-08 15:13:28 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-02-25 11:03:40 +0100 |
| commit | a7fbdc8875a676b9ddef27b0149b4a686725fca8 (patch) | |
| tree | 679f47b6f8ab27832c7817488460aecce94ffcf1 /lib/interval_tree_test.c | |
| parent | 6fdf5856c6a513a6b2ea37c0fed2b1450e611be6 (diff) | |
509: fix printing uninitialized stack memory when OID is empty
[ Upstream commit 8dfd2f22d3bf3ab7714f7495ad5d897b8845e8c1 ]
Callers of sprint_oid() do not check its return value before printing
the result. In the case where the OID is zero-length, -EBADMSG was
being returned without anything being written to the buffer, resulting
in uninitialized stack memory being printed. Fix this by writing
"(bad)" to the buffer in the cases where -EBADMSG is returned.
Fixes: 4f73175d0375 ("X.509: Add utility functions to render OIDs as strings")
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'lib/interval_tree_test.c')
0 files changed, 0 insertions, 0 deletions