commit e7033c8a901bdbb2356ddb38cb849b995140acc7
author Lei Zhang <thestig@chromium.org> Tue Feb 26 19:30:49 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Feb 26 19:30:49 2019 +0000
tree 9b911d1eb3a0925727c2eedc4b9bdb9a45d5ca2a
parent d94d69ae98c498c15bdb579332a62679cded438d

Test bad inputs for newly added annotation APIs.

Change-Id: I3297ebf5c8ee71baaefb128a7d68bbcb699866a5
Reviewed-on: https://pdfium-review.googlesource.com/c/51153
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>

fpdfsdk/fpdf_annot_embeddertest.cpp

diff --git a/fpdfsdk/fpdf_annot_embeddertest.cpp b/fpdfsdk/fpdf_annot_embeddertest.cpp
index 9a3e681..e9ed205 100644
--- a/fpdfsdk/fpdf_annot_embeddertest.cpp
+++ b/fpdfsdk/fpdf_annot_embeddertest.cpp
@@ -1690,6 +1690,11 @@
     ASSERT_TRUE(annot);
 
     EXPECT_EQ(26, FPDFAnnot_GetOptionCount(form_handle(), annot.get()));
+
+    // Check bad form handle / annot.
+    EXPECT_EQ(-1, FPDFAnnot_GetOptionCount(nullptr, nullptr));
+    EXPECT_EQ(-1, FPDFAnnot_GetOptionCount(form_handle(), nullptr));
+    EXPECT_EQ(-1, FPDFAnnot_GetOptionCount(nullptr, annot.get()));
   }
 
   UnloadPage(page);
@@ -1777,11 +1782,18 @@
                                             buf.data(), len));
     EXPECT_STREQ(L"Zucchini", BufferToWString(buf).c_str());
 
-    // indices out of range
+    // Indices out of range
     EXPECT_EQ(0u, FPDFAnnot_GetOptionLabel(form_handle(), annot.get(), -1,
                                            nullptr, 0));
     EXPECT_EQ(0u, FPDFAnnot_GetOptionLabel(form_handle(), annot.get(), 26,
                                            nullptr, 0));
+
+    // Check bad form handle / annot.
+    EXPECT_EQ(0u, FPDFAnnot_GetOptionLabel(nullptr, nullptr, 0, nullptr, 0));
+    EXPECT_EQ(0u,
+              FPDFAnnot_GetOptionLabel(nullptr, annot.get(), 0, nullptr, 0));
+    EXPECT_EQ(0u,
+              FPDFAnnot_GetOptionLabel(form_handle(), nullptr, 0, nullptr, 0));
   }
 
   UnloadPage(page);