Commit 12d1ead

committed

Add payload length validation to ping and pong

1 parent 206d2d5 commit 12d1eadCopy full SHA for 12d1ead

File tree

-2

lines changed

-2

lines changed

Lines changed: 8 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -499,8 +499,11 @@ public void run() {`
`499`	`499`	`* @param data Data to be sent, or null if there is no data.`
`500`	`500`	`*/`
`501`	`501`	`public void sendPing(byte[] data) {`
`502`		`- final Payload payload = new Payload(OPCODE_PING, data);`
	`502`	`+ if (data != null && data.length > 125) {`
	`503`	`+ throw new IllegalArgumentException("Control frame payload cannot be greater than 125 bytes");`
	`504`	`+ }`
`503`	`505`
	`506`	`+ final Payload payload = new Payload(OPCODE_PING, data);`
`504`	`507`	`new Thread(new Runnable() {`
`505`	`508`	`@Override`
`506`	`509`	`public void run() {`
`@@ -515,8 +518,11 @@ public void run() {`
`515`	`518`	`* @param data Data to be sent, or null if there is no data.`
`516`	`519`	`*/`
`517`	`520`	`public void sendPong(byte[] data) {`
`518`		`- final Payload payload = new Payload(OPCODE_PONG, data);`
	`521`	`+ if (data != null && data.length > 125) {`
	`522`	`+ throw new IllegalArgumentException("Control frame payload cannot be greater than 125 bytes");`
	`523`	`+ }`
`519`	`524`
	`525`	`+ final Payload payload = new Payload(OPCODE_PONG, data);`
`520`	`526`	`new Thread(new Runnable() {`
`521`	`527`	`@Override`
`522`	`528`	`public void run() {`

Comments

(0)