commit 972804b81edb242716d6fb1b57c9584832b4e8e1
author Jooyung Han <jooyung@google.com> Fri Jun 21 22:41:16 2024 +0000
committer Jooyung Han <jooyung@google.com> Fri Jun 21 22:42:48 2024 +0000
tree a7a5d6149aefe45474a8c7432ab26de7e4a6ee18
parent a95db664df019c76a0e453a4d24074ca82043f2a

Fix a bug in the releasetools common.py script.

The script was not breaking out of the loop when a handler was found, which could lead to multiple handlers being called for the same option.

This fixes the issue that a single option can be handled in two handlers.

Bug: 345402175
Test: sign_target_files_apks -k ..
Change-Id: I9f4ef801df99a4a0f2ba5be994b3f37fbedc22c0

tools/releasetools/common.py

diff --git a/tools/releasetools/common.py b/tools/releasetools/common.py
index a949c11..4834834 100644
--- a/tools/releasetools/common.py
+++ b/tools/releasetools/common.py
@@ -2788,6 +2788,7 @@
             break
         elif handler(o, a):
           success = True
+          break
       if not success:
         raise ValueError("unknown option \"%s\"" % (o,))