Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(836)

Issue 1689873003: Fix PEM error checking (Closed)

Created:
4 years, 10 months ago by zra
Modified:
4 years, 10 months ago
Reviewers:
Bill Hesse, Ivan Posva
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix PEM error checking While reading PEM data, if we encounter PEM_R_NO_START_LINE, and some data has been read successfully, we assume that we have reached the end of the data. If we encounter PEM_R_NO_START_LINE, and no data has been read successfully, we assume that the data is not PEM, and we try instead to read it as PKCS12. Otherwise, we assume it is malformed PEM, and propagate an error. R=whesse@google.com Committed: https://github.com/dart-lang/sdk/commit/53dec5d35fae91becee40b635a7ff8cda9edf9d9

Patch Set 1 #

Patch Set 2 : Remove unused field from SSLFilter #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -23 lines) Patch
M runtime/bin/secure_socket.h View 1 1 chunk +0 lines, -1 line 0 comments Download
M runtime/bin/secure_socket.cc View 8 chunks +31 lines, -22 lines 0 comments Download
M tests/standalone/io/security_context_argument_test.dart View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
zra
4 years, 10 months ago (2016-02-11 07:58:38 UTC) #2
zra
Ping
4 years, 10 months ago (2016-02-12 15:57:38 UTC) #3
zra
4 years, 10 months ago (2016-02-12 15:57:50 UTC) #5
Bill Hesse
lgtm
4 years, 10 months ago (2016-02-12 23:45:30 UTC) #6
zra
4 years, 10 months ago (2016-02-13 03:01:52 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
53dec5d35fae91becee40b635a7ff8cda9edf9d9 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698