Fix: Correctly convert 'PInt' value to byte.

Author: fangerer

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/common/SequenceStorageNodes.java

@@ -40,6 +40,7 @@
 import com.oracle.graal.python.nodes.expression.BinaryComparisonNode;
 import com.oracle.graal.python.nodes.expression.CastToBooleanNode;
 import com.oracle.graal.python.nodes.util.CastToIndexNode;
+import com.oracle.graal.python.runtime.exception.PException;
 import com.oracle.graal.python.runtime.object.PythonObjectFactory;
 import com.oracle.graal.python.runtime.sequence.PSequence;
 import com.oracle.graal.python.runtime.sequence.storage.BasicSequenceStorage;
@@ -826,7 +827,7 @@ protected byte doIntOvf(int value) {
             try {
                 return PInt.byteValueExact(value);
             } catch (ArithmeticException e) {
-                throw raise(ValueError, "byte must be in range(0, 256)");
+                throw raiseByteRangeError();
             }
         }
 
@@ -840,21 +841,21 @@ protected byte doLongOvf(long value) {
             try {
                 return PInt.byteValueExact(value);
             } catch (ArithmeticException e) {
-                throw raise(ValueError, "byte must be in range(0, 256)");
+                throw raiseByteRangeError();
             }
         }
 
         @Specialization(rewriteOn = ArithmeticException.class)
         protected byte doPInt(PInt value) {
-            return value.byteValueExact();
+            return PInt.byteValueExact(value.longValueExact());
         }
 
         @Specialization(replaces = "doPInt")
         protected byte doPIntOvf(PInt value) {
             try {
-                return value.byteValueExact();
+                return PInt.byteValueExact(value.longValueExact());
             } catch (ArithmeticException e) {
-                throw raise(ValueError, "byte must be in range(0, 256)");
+                throw raiseByteRangeError();
             }
         }
 
@@ -868,6 +869,10 @@ protected byte doGeneric(@SuppressWarnings("unused") Object val) {
             throw raise(TypeError, "an integer is required");
         }
 
+        private PException raiseByteRangeError() {
+            throw raise(ValueError, "byte must be in range(0, 256)");
+        }
+
         public static CastToByteNode create() {
             return CastToByteNodeGen.create();
         }