safebox/guacamole-client
4
0
Fork 0
mirror of https://github.com/gyurix1968/guacamole-client.git synced 2025-09-06 13:17:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #321 from glyptodon/GUAC-1463

Browse Source 
GUAC-1463: Don't trust the keydown event if it's for a printable character.
This commit is contained in:
Mike Jumper
2016-01-18 20:30:51 -08:00
parent 7a0c1fc785 f4e94e22f0
commit 3c35437627
1 changed files with 22 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
guacamole-common-js/src/main/webapp/modules/Keyboard.js
View File

@@ -178,8 +178,9 @@ Guacamole.Keyboard = function(element) {
this.keysym = keysym_from_key_identifier(key, location)
|| keysym_from_keycode(keyCode, location);
// DOM3 and keyCode are reliable sources
if (this.keysym)
// DOM3 and keyCode are reliable sources if the corresponding key is
// not a printable key
if (this.keysym && !isPrintable(this.keysym))
this.reliable = true;
// Use legacy keyIdentifier as a last resort, if it looks sane
@@ -591,6 +592,25 @@ Guacamole.Keyboard = function(element) {
return keysyms[location] || keysyms[0];
};
/**
* Returns true if the given keysym corresponds to a printable character,
* false otherwise.
*
* @param {Number} keysym
* The keysym to check.
*
* @returns {Boolean}
* true if the given keysym corresponds to a printable character,
* false otherwise.
*/
var isPrintable = function isPrintable(keysym) {
// Keysyms with Unicode equivalents are printable
return (keysym >= 0x00 && keysym <= 0xFF)
|| (keysym & 0xFFFF0000) === 0x01000000;
};
function keysym_from_key_identifier(identifier, location, shifted) {
if (!identifier)