commit be03a851e438a367ab23005a1117d6397ba8aae1
author Greg Kaiser <gkaiser@google.com> Mon Feb 11 11:20:39 2019 -0800
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Feb 12 13:50:10 2019 +0000
tree 72dfbfaaa36e05130e541e5ffe496858b02442e7
parent 3999f1076247c1a6c602d32110d6f6f833bee70a

SkUTF: Avoid potential nullptr dereference

next_fail() dereferences the first argument.  So calling
next_fail() with the first argument of 'ptr' when we know 'ptr'
is nullptr will cause a problem.

For NextUTF16(), we instead return -1 when we know 'ptr' is
nullptr, just as the code currently does for NextUTF8() and
NextUTF32().

Change-Id: Ifabb45c694d6e3dbeee5ac7b953f15895cea3a69
Reviewed-on: https://skia-review.googlesource.com/c/191420
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>