Fix UBSan undefined behavior warning

moonlight-ios/Limelight/Input/ControllerSupport.m:294:32: runtime error: 255 is outside the range of representable values of type 'char'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior moonlight-ios/Limelight/Input/ControllerSupport.m:294:32 in
This commit is contained in:
Cameron Gutman
2019-11-06 20:57:53 -08:00
parent 2577f581ba
commit 5a0e4bcc05
2 changed files with 7 additions and 7 deletions
+3 -3
View File
@@ -28,9 +28,9 @@
-(void) updateLeftStick:(Controller*)controller x:(short)x y:(short)y; -(void) updateLeftStick:(Controller*)controller x:(short)x y:(short)y;
-(void) updateRightStick:(Controller*)controller x:(short)x y:(short)y; -(void) updateRightStick:(Controller*)controller x:(short)x y:(short)y;
-(void) updateLeftTrigger:(Controller*)controller left:(char)left; -(void) updateLeftTrigger:(Controller*)controller left:(unsigned char)left;
-(void) updateRightTrigger:(Controller*)controller right:(char)right; -(void) updateRightTrigger:(Controller*)controller right:(unsigned char)right;
-(void) updateTriggers:(Controller*)controller left:(char)left right:(char)right; -(void) updateTriggers:(Controller*)controller left:(unsigned char)left right:(unsigned char)right;
-(void) updateButtonFlags:(Controller*)controller flags:(int)flags; -(void) updateButtonFlags:(Controller*)controller flags:(int)flags;
-(void) setButtonFlag:(Controller*)controller flags:(int)flags; -(void) setButtonFlag:(Controller*)controller flags:(int)flags;
+4 -4
View File
@@ -91,21 +91,21 @@
} }
} }
-(void) updateLeftTrigger:(Controller*)controller left:(char)left -(void) updateLeftTrigger:(Controller*)controller left:(unsigned char)left
{ {
@synchronized(controller) { @synchronized(controller) {
controller.lastLeftTrigger = left; controller.lastLeftTrigger = left;
} }
} }
-(void) updateRightTrigger:(Controller*)controller right:(char)right -(void) updateRightTrigger:(Controller*)controller right:(unsigned char)right
{ {
@synchronized(controller) { @synchronized(controller) {
controller.lastRightTrigger = right; controller.lastRightTrigger = right;
} }
} }
-(void) updateTriggers:(Controller*) controller left:(char)left right:(char)right -(void) updateTriggers:(Controller*) controller left:(unsigned char)left right:(unsigned char)right
{ {
@synchronized(controller) { @synchronized(controller) {
controller.lastLeftTrigger = left; controller.lastLeftTrigger = left;
@@ -248,7 +248,7 @@
Controller* limeController = [self->_controllers objectForKey:[NSNumber numberWithInteger:gamepad.controller.playerIndex]]; Controller* limeController = [self->_controllers objectForKey:[NSNumber numberWithInteger:gamepad.controller.playerIndex]];
short leftStickX, leftStickY; short leftStickX, leftStickY;
short rightStickX, rightStickY; short rightStickX, rightStickY;
char leftTrigger, rightTrigger; unsigned char leftTrigger, rightTrigger;
UPDATE_BUTTON_FLAG(limeController, A_FLAG, gamepad.buttonA.pressed); UPDATE_BUTTON_FLAG(limeController, A_FLAG, gamepad.buttonA.pressed);
UPDATE_BUTTON_FLAG(limeController, B_FLAG, gamepad.buttonB.pressed); UPDATE_BUTTON_FLAG(limeController, B_FLAG, gamepad.buttonB.pressed);