fix 1088: correct BooleanObject compare

py-pdf · Jul 25, 2022 · 8661b1a · 8661b1a
1 parent 0b27287
commit 8661b1a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/PyPDF2/_encryption.py b/PyPDF2/_encryption.py
@@ -288,7 +288,7 @@ def compute_key(
         u_hash.update(o_entry)
         u_hash.update(struct.pack("<I", P))
         u_hash.update(id1_entry)
-        if rev >= 3 and not metadata_encrypted:
+        if rev >= 4 and metadata_encrypted == False:
             u_hash.update(b"\xff\xff\xff\xff")
         u_hash_digest = u_hash.digest()
         length = key_size // 8

diff --git a/tests/test_page.py b/tests/test_page.py
@@ -449,7 +449,6 @@ def test_text_extraction_issue_1091():
         page.extract_text()
 
 
-@pytest.mark.xfail(reason="#1088")
 def test_empyt_password_1088():
     url = "https://corpora.tika.apache.org/base/docs/govdocs1/941/941536.pdf"
     name = "tika-941536.pdf"