| commit | 8bad7375d7e0ebe28af4d905665f34227843bd16 | [log] [tgz] |
|---|---|---|
| author | edisonn@google.com <edisonn@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | Wed Jul 10 23:36:56 2013 +0000 |
| committer | edisonn@google.com <edisonn@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | Wed Jul 10 23:36:56 2013 +0000 |
| tree | 45851e2527f73a713b48dc27fc9d6582fb03f20e | |
| parent | c8b3037eeb15936285767ed07528efb4263edaaa [diff] [blame] |
pdfviewer: ignore more bad stauff in pdfs Review URL: https://codereview.chromium.org/18112023 git-svn-id: http://skia.googlecode.com/svn/trunk@9990 2bbb7eff-a529-9590-31e7-b0007b416f81
diff --git a/experimental/PdfViewer/pdfparser/native/SkPdfNativeTokenizer.cpp b/experimental/PdfViewer/pdfparser/native/SkPdfNativeTokenizer.cpp index 1288132..604e9fb 100644 --- a/experimental/PdfViewer/pdfparser/native/SkPdfNativeTokenizer.cpp +++ b/experimental/PdfViewer/pdfparser/native/SkPdfNativeTokenizer.cpp
@@ -69,7 +69,8 @@ array->appendInArray(newObj); } // TODO(edisonn): report not reached, we should never get here - SkASSERT(false); + // TODO(edisonn): there might be a bug here, enable an assert and run it on files + // or it might be that the files were actually corrupted return start; } @@ -170,6 +171,8 @@ in++; break; } + } else { + in++; } } else { // TODO(edisonn): perf, avoid copy into itself, maybe first do a simple scan until found backslash ?