1. 15 May, 2017 5 commits
  2. 13 May, 2017 1 commit
    • Cedric Roux's avatar
      rewrite phy_stats_exist · 9eaee53b
      Cedric Roux authored
      We have to test that all active CCs of a given UE exist in the PHY layer.
      Maybe not the end of the story...
      9eaee53b
  3. 12 May, 2017 14 commits
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Cedric Roux's avatar
    • Tien-Thinh Nguyen's avatar
      fix the errors related to multiple carriers (for a given UE considering only... · 3d4ba1a3
      Tien-Thinh Nguyen authored
      fix the errors related to multiple carriers (for a given UE considering only the CCs which this UE belongs to)
      3d4ba1a3
    • gabrielC's avatar
      bug fixes from Fujitsu (bug 37) · ad6d0ac5
      gabrielC authored
      ```-------------------------------------------------------
      bug 37
      
      Ttile:
      Sending side is as follows.
      rar[3] = (((mcs&0x7)<<5)) | ((TPC&7)<<2) | ((ULdelay&1)<<1) | (cqireq&1);
      So, 2 bit shift looks correct.
      
      Bug Location:
      ulsch->harq_processes[harq_pid]->TPC = (rar[3]>>3)&7;//rar->TPC;
      ```
      
      -------------------------------------------------------
      ad6d0ac5
    • gabrielC's avatar
      bug fixes from Fujitsu (bug 29) · 893f36e8
      gabrielC authored
      ```-------------------------------------------------------
      bug 29
      
      Ttile:
      FDD should be TDD.
      
      Bug Location:
        switch (oai_emulation.info.frame_type[0]) {
        case FDD:
          frame_type = "FDD";
          break;
      
        case TDD:
          frame_type = "FDD";
          break;
        }
      ```
      
      -------------------------------------------------------
      893f36e8
    • gabrielC's avatar
      bug fixes from Fujitsu (bug 26) · 899d1506
      gabrielC authored
      ```-------------------------------------------------------
      bug 26
      
      Ttile:
      g_buffer has only 32 index, but he/she tries to zero clear
      1024 index.
      
      Bug Location:
      static uint8_t g_buffer[32];
      :
      memset (g_buffer, 0, 1024);
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      899d1506
    • Cedric Roux's avatar
      bug fixes from Fujitsu (bug 24) · adb3b25c
      Cedric Roux authored
      ```-------------------------------------------------------
      bug 24
      
      Ttile:
      The last line "multicast_group = multicast_group" is hard
      to understand. The local variable should be different name
      than the global variable.
      
      Bug Location:
      const char *multicast_group_list[MULTICAST_LINK_NUM_GROUPS] = {
        "239.0.0.161",
        "239.0.0.162",
        "239.0.0.163",
        "239.0.0.164"
      };
      :
      :
      void multicast_link_start(void (*rx_handlerP) (unsigned int, char *),
                                unsigned char multicast_group, char *multicast_ifname)
      {
        rx_handler = rx_handlerP;
        multicast_group = multicast_group;
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      adb3b25c
    • gabrielC's avatar
      bug fixes from Fujitsu (bug 22) · 34c6805f
      gabrielC authored
      Note: not sure that the code is correct,
      to be checked (Cédric Roux).
      
      ----------------------------------------------------------
      bug 22
      
      Ttile:
      (subframe!=2)||(subframe!=7) is always TRUE. This coding
      is really intended? Operator || should be && instead?
      
      Bug Location:
      ((frame_parms->frame_type==TDD)&&(frame_parms->tdd_config==1)&&((subframe!=2)||(subframe!=7)))) {
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      34c6805f
    • gabrielC's avatar
      bug fixes from Fujitsu (bug 21) · 98a7a16e
      gabrielC authored
      Note: this fix does not solve the issue with how
      this buffer is used in this function, but I'm not
      sure this code is used at all (Cédric Roux).
      
      ----------------------------------------------------------
      bug 21
      
      Ttile:
      PAYLOAD_MAX=1500, so this coding results in index overflow.
      Additionally, operator "!=" just compairs something, this
      coding is meaningless.
      
      Bug Location:
      buffer[PAYLOAD_MAX] != '\0';
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      98a7a16e
    • gabrielC's avatar
      bug fixes from Fujitsu (bugs 18, 19, 20) · bdaef795
      gabrielC authored
      ```-------------------------------------------------------
      bug 18
      
      Ttile:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      u2e_t[UE_id][eNB_id]->tx_data[3]=tx[eNB_id][3];
      u2e_t[UE_id][eNB_id]->rx_data[3]=rx[NB_eNB_INST+UE_id][3];
      
      Note:
      Detected by CppCheck
      ```
      
      -------------------------------------------------------
      bug 19
      
      Title:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      e2u_t[eNB_id][UE_id]->tx_data[3]=tx[NB_eNB_INST+UE_id][3];
      e2u_t[eNB_id][UE_id]->rx_data[3]=rx[eNB_id][3];
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      bug 20
      
      Title:
      Probably he/she wants to copy the contents of arrays, but
      this coding doesn't copy anything, moreover index overflow
      occurs.
      
      Bug Location:
      tx_data[3]=cthread->tx_data[3];
      rx_data[3]=cthread->rx_data[3];
      
      Note:
      Detected by CppCheck
      ----------------------------------------------------------
      bdaef795
    • Francois TABURET's avatar
      Fix shared lib build problem, where shared lib sources were also included in main exec's · 26135368
      Francois TABURET authored
      Discovered and fix problems in shared lib build for usrp,bladerf and lime
      26135368
  4. 03 May, 2017 6 commits
  5. 02 May, 2017 1 commit
  6. 26 Apr, 2017 1 commit
  7. 25 Apr, 2017 3 commits
  8. 24 Apr, 2017 2 commits
  9. 21 Apr, 2017 2 commits
  10. 18 Apr, 2017 4 commits
  11. 14 Apr, 2017 1 commit
    • Cedric Roux's avatar
      hotfix: fix TDD 20MHz DCI0 structure · 20a2bfd1
      Cedric Roux authored
      In 20MHz the UE didn't do any uplink granted by DCI0.
      It was replying to RAR, so uplink decoding was okay.
      
      Turns out the DCI0 structure for TDD was wrong.
      20a2bfd1