Skip to content
Snippets Groups Projects
  1. May 30, 2016
  2. May 28, 2016
  3. May 26, 2016
    • est31's avatar
      Tell irrlicht if we handle a key or not. · fa6b21a1
      est31 authored
      We can remove the function in MtNativeActivity now
      as it serves precisely that purpose: to tell irrlicht
      that we handled the esc key.
      
      TODO for later:
       * Perhaps try to find a more performant container than KeyList
      fa6b21a1
  4. May 23, 2016
  5. May 22, 2016
    • est31's avatar
      Tolerate packet reordering in the early init process · 423d8c1b
      est31 authored
      Fixes a bug where packet reordering made the server give the
      client two peer ids instead of one. This in turn confused
      reliable packet sending and made connecting to the server fail.
      
      The client usually sends three packets at init: one "dummy"
      packet consisting of two 0 bytes, and the init packet as well as
      its legacy counterpart. The last one can be turned off since commit
      af301831, but this is of lower
      relevance for the bug. The relevant part here is that network
      packet reorder (which is a normal occurence) can make the packets
      reach the server in different order.
      
      If reorder puts the dummy packet further behind, the following
      would happen before the patch:
      
      1. The server will get one of the init packets on channel 1 and
         assign the client a peer id, as the packet will have zero as
         peer id.
      
      2. The server sends a CONTROLTYPE_SET_PEER_ID packet to inform
         the client of the peer id.
      
      3. The next packet from the client will contain the peer id set by
         the server.
      
      4. The server sets the m_has_sent_with_id member for the client's
         peer structure to true.
      
      5. Now the dummy packet arrives. It has a peer id of zero, therefore
         the server searches whether it already has a peer id for the
         address the packet was sent from. The search fails because
         m_has_sent_with_id was set to true and the server only searched
         for peers with m_has_sent_with_id set to false.
      
      6. In a working setup, the server would assign the dummy packet to
         the correct peer id. However the server instead now assigns a
         second peer id and peer structure to the peer, and assign the
         packet to that new peer.
      
      7. In order to inform the peer of its peer id, the server sends a
         CONTROLTYPE_SET_PEER_ID command packet, reliably, to the peer.
         This packet uses the new peer id.
      
      8. The client sends an ack to that packet, not with the new peer id
         but with the peer id sent in 2.
      
      9. This packet reaches the server, but it drops the ACK as the peer
         id does not map to any un-ACK-ed packets with that seqnum. The
         same time, the server still waits for an ACK with the new peer
         id, which of course won't come. This causes the server to
         periodically re-try sending that packet, and the client ACKing it
         each time.
      
      Steps 7-9 cause annoyances and erroneous output, but don't cause
      the connection failure itself.
      The actual mistake that causes the connection failure happens in 6:
      The server does not assign the dummy packet to the correct peer, but
      to a newly created one.
      Therefore, all further packets sent by the client on channel 0 are
      now buffered by the server as it waits for the dummy packet to reach
      the peer, which of course doesn't happen as the server assigned
      that packet to the second peer it created for the client.
      This makes the connection code indefinitely buffer the
      TOSERVER_CLIENT_READY packet, not passing it to higher level code,
      which stalls the continuation of the further init process
      indefinitely and causes the actual bug.
      
      Maybe this can be caused by reordered init packets as well, the only
      studied case was where network has reliably reordered the dummy
      packet to get sent after the init packets.
      
      The patch fixes the bug by not ignoring peers where
      m_has_sent_with_id has been set anymore. The other changes of the
      patch are just cleanups of unused methods and fields and additional
      explanatory comments.
      
      One could think of alternate ways to fix the bug:
      
      * The client could simply take the new peer id and continue
        communicating with that. This is however worse than the fix as
        it requires the peer id set command to be sent reliably (which
        currently happens, but it cant be changed anymore). Also, such a
        change would require both server and client to be patched in order
        for the bug to be fixed, as right now the client ignores peer id
        set commands after the peer id is different from
        PEER_ID_INEXISTENT and the server requires modification too to
        change the peer id internally.
        And, most importantly, right now we guarantee higher level server
        code that the peer id for a certain peer does not change. This
        guarantee would have to be broken, and it would require much
        larger changes to the server than this patch means.
      
      * One could stop sending the dummy packet. One may be unsure whether
        this is a good idea, as the meaning of the dummy packet is not
        known (it might be there for something important), and as it is
        possible that the init packets may cause this problem as well
        (although it may be possible too that they can't cause this).
      
      Thanks to @auouYmous who had originally reported this bug and who
      has helped patiently in finding its cause.
      423d8c1b
    • Loïc Blot's avatar
      f64a6259
    • Loïc Blot's avatar
      Implement a PostgreSQL backend · ce42ff9c
      Loïc Blot authored
      ce42ff9c
    • HybridDog's avatar
      Gitignore: ignore idea and ninja files · 0f184d77
      HybridDog authored
      0f184d77
    • paramat's avatar
      Item entities: Don't show description as infotext · 643ac9dd
      paramat authored
      Partially reverts #3547
      Infotext remains optional for objects, empty by default
      643ac9dd
  6. May 20, 2016
    • Zeno-'s avatar
      Fix tooltip height for versions of irrlicht < 1.8.2 · 88acda02
      Zeno- authored
      Version 1.8.2 of irrlicht changed the way that IGUIStaticText::getTextHeight() works and since that release properly deals with newlines.
      
      From irrlicht changes.txt for 1.8.2, "IGUIStaticText::getTextHeight returns now the correct height for texts with newlines even WordWrap is not set."
      88acda02
Loading