BufferedOutputStream не записывает все в файл

Question

Я пытаюсь создать TFTP-сервер, но когда он получает файл, кажется, что не все его сохраняется на сервере (некоторые байты отсутствуют). Файл создается отлично, и большая часть данных записывается, но поскольку файл не является полным, он классифицируется как поврежденный и неприменимый. Кто-нибудь знает, как решить эту проблему?

главный класс

  WRQ WRQ = new WRQ(); 
      ACK ACK = new ACK(); 
      DatagramPacket outPacket; 
      BufferedOutputStream bufferedOutput = new BufferedOutputStream(new FileOutputStream(filename)); 
      byte[] bytes; 
      byte[] fileOut; 
      outPacket = WRQ.firstPacket(packet); 
      socket.send(outPacket); 

      socket.receive(packet); 

      while (packet.getLength() == 516){ 

      bytes = WRQ.doWRQ(packet); 
      bufferedOutput.write(bytes); 

      outPacket = ACK.doACK(packet); 
      socket.send(outPacket); 

      socket.receive(packet); 

      } 

      bytes = WRQ.doWRQ(packet); 
      bufferedOutput.write(bytes); 

      bufferedOutput.close(); 

      outPacket = ACK.doACK(packet); 
      socket.send(outPacket); 

WRQ класс

public class WRQ { 

public DatagramPacket firstPacket(DatagramPacket packet) throws IOException{ 

    ACK ACK = new ACK(); 
    DatagramPacket ACKpacket = ACK.doACK(packet); 

    //takes ACK packet and sets block # as 0 to signal that this is the first packet in a WRQ 
    byte[] ACKcontents = new byte[3]; 
    ACKcontents = ACKpacket.getData(); 
    ACKcontents[2] = 0; 
    ACKcontents[3] = 0; 
    ACKpacket.setData(ACKcontents); 

    return ACKpacket; 

} 

public byte[] doWRQ(DatagramPacket packet){ 

    int length = packet.getLength(); 
    byte[] packetData = packet.getData(); 
    byte[] data = new byte[length - 4]; 
    data = Arrays.copyOfRange(packetData, 4, length - 4); 

    return data; 

} 

} 

Answer 1

Этот код выглядит очень подозрительно ко мне:

byte[] packetData = packet.getData(); 
byte[] data = new byte[length - 4]; 
data = Arrays.copyOfRange(packetData, 4, length - 4); 

Ваш выходной массив (данных) имеет длину length - 4, но вы копировать только length - 8 байт. Если байты игнорирующие в packetData первые 4 байта, он должен быть

data = Arrays.copyOfRange(packetData, 4, length); 

потому, что последний аргумент не длина, а to индекс (эксклюзив). См. the javadoc.