bubgum / crw-cmu

Automatically exported from code.google.com/p/crw-cmu
0 stars 0 forks source link

UdpVehicleServer.captureImage creates oversized UDP packets #58

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Connect to a UDPVehicleService using UDPVehicleServer
2. Call captureImage with a 640x480 image
3. Watch the exceptions on the UDPVehicleService

What is the expected output? What do you see instead?

Mar 21, 2012 5:05:16 PM edu.cmu.ri.crw.udp.UdpServer respond
WARNING: Failed to respond.
java.io.IOException: Message too long
    at java.net.PlainDatagramSocketImpl.send(Native Method)
    at java.net.DatagramSocket.send(DatagramSocket.java:675)
    at edu.cmu.ri.crw.udp.UdpServer.respond(UdpServer.java:411)
    at edu.cmu.ri.crw.udp.UdpVehicleService.received(UdpVehicleService.java:181)
    at edu.cmu.ri.crw.udp.UdpServer$Receiver.run(UdpServer.java:320)
    at java.lang.Thread.run(Thread.java:722)

Please use labels and text to provide additional information.

captureImage is not using the image retransmission table that startCapture and 
stopCapture use to split and reassemble their images.  As a result, it creates 
a single UDP packet containing the entire image, which is larger than UDP spec 
for most images.

Original issue reported on code.google.com by psi...@gmail.com on 21 Mar 2012 at 9:07