[jsinterp] Fix for youtube player c81bbb4a

test/test_jsinterp.py

@@ -212,6 +212,11 @@ def test_comma(self):
         ''')
         self.assertEqual(jsi.call_function('x'), 7)
 
+        jsi = JSInterpreter('''
+        function x() { return (l=[0,1,2,3], function(a, b){return a+b})((l[1], l[2]), l[3]) }
+        ''')
+        self.assertEqual(jsi.call_function('x'), 5)
+
     def test_void(self):
         jsi = JSInterpreter('''
         function x() { return void 42; }

test/test_youtube_signature.py

@@ -102,6 +102,10 @@
         'https://www.youtube.com/s/player/4c3f79c5/player_ias.vflset/en_US/base.js',
         'TDCstCG66tEAO5pR9o', 'dbxNtZ14c-yWyw',
     ),
+    (
+        'https://www.youtube.com/s/player/c81bbb4a/player_ias.vflset/en_US/base.js',
+        'gre3EcLurNY2vqp94', 'Z9DfGxWP115WTg',
+    ),
 ]
 
 

yt_dlp/jsinterp.py

@@ -33,19 +33,19 @@
     '==': operator.eq,
     '!=': operator.ne,
 
-    '<=': operator.le,
-    '>=': operator.ge,
-    '<': operator.lt,
-    '>': operator.gt,
+    '<=': lambda a, b: (a or 0) <= (b or 0),
+    '>=': lambda a, b: (a or 0) >= (b or 0),
+    '<': lambda a, b: (a or 0) < (b or 0),
+    '>': lambda a, b: (a or 0) > (b or 0),
 
     '>>': operator.rshift,
     '<<': operator.lshift,
 
-    '+': operator.add,
-    '-': operator.sub,
+    '+': lambda a, b: (a or 0) + (b or 0),
+    '-': lambda a, b: (a or 0) - (b or 0),
 
-    '*': operator.mul,
-    '/': operator.truediv,
+    '*': lambda a, b: (a or 0) * (b or 0),
+    '/': lambda a, b: (a or 0) / b,
     '%': operator.mod,
 
     '**': operator.pow,
@@ -339,11 +339,12 @@ def interpret_statement(self, stmt, local_vars, allow_recursion=100):
 
         # Comma separated statements
         sub_expressions = list(self._separate(expr))
-        expr = sub_expressions.pop().strip() if sub_expressions else ''
-        for sub_expr in sub_expressions:
-            ret, should_abort = self.interpret_statement(sub_expr, local_vars, allow_recursion)
-            if should_abort:
-                return ret, True
+        if len(sub_expressions) > 1:
+            for sub_expr in sub_expressions:
+                ret, should_abort = self.interpret_statement(sub_expr, local_vars, allow_recursion)
+                if should_abort:
+                    return ret, True
+            return ret, False
 
         for m in re.finditer(rf'''(?x)
                 (?P<pre_sign>\+\+|--)(?P<var1>{_NAME_RE})|
@@ -422,8 +423,7 @@ def interpret_statement(self, stmt, local_vars, allow_recursion=100):
             if not separated:
                 continue
             left_val = self.interpret_expression(op.join(separated), local_vars, allow_recursion)
-            return self._operator(op, 0 if left_val is None else left_val,
-                                  right_expr, expr, local_vars, allow_recursion), should_return
+            return self._operator(op, left_val, right_expr, expr, local_vars, allow_recursion), should_return
 
         if m and m.group('attribute'):
             variable = m.group('var')