Commit0dc15a9

committed

js: Fix externally-controlled format strings

CodeQL is now complaining about these. This should be okay since we onlytalk to ourselves, but better to be safe about it.

1 parentcf6069a commit0dc15a9Copy full SHA for 0dc15a9

File tree

-3

lines changed

-3

lines changed

Lines changed: 5 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -575,19 +575,21 @@ mpl.figure.prototype._make_on_message_function = function (fig) {`
`575`	`575`	`varcallback=fig['handle_'+msg_type];`
`576`	`576`	`}catch(e){`
`577`	`577`	`console.log(`
`578`		`-"No handler for the '"+msg_type+"' message type: ",`
	`578`	`+"No handler for the '%s' message type: ",`
	`579`	`+msg_type,`
`579`	`580`	`msg`
`580`	`581`	`);`
`581`	`582`	`return;`
`582`	`583`	`}`
`583`	`584`
`584`	`585`	`if(callback){`
`585`	`586`	`try{`
`586`		`-// console.log("Handling '" + msg_type + "' message: ", msg);`
	`587`	`+// console.log("Handling '%s' message: ", msg_type, msg);`
`587`	`588`	`callback(fig,msg);`
`588`	`589`	`}catch(e){`
`589`	`590`	`console.log(`
`590`		`-"Exception inside the 'handler_"+msg_type+"' callback:",`
	`591`	`+"Exception inside the 'handler_%s' callback:",`
	`592`	`+msg_type,`
`591`	`593`	`e,`
`592`	`594`	`e.stack,`
`593`	`595`	`msg`

Comments

(0)