1. 09 Mar, 2018 2 commits
  2. 26 Feb, 2018 1 commit
  3. 23 Feb, 2018 1 commit
  4. 09 Feb, 2018 2 commits
  5. 02 Feb, 2018 1 commit
  6. 19 Jan, 2018 1 commit
  7. 17 Jan, 2018 1 commit
  8. 29 Nov, 2017 2 commits
  9. 20 Nov, 2017 1 commit
  10. 11 Nov, 2017 1 commit
    • Romain Vimont's avatar
      gui/qt: bookmarks: fix psz_name lifetime · ea0a32e1
      Romain Vimont authored
      The macro qtu() is defined as follows:
      
          #define qtu( i ) ((i).toUtf8().constData())
      
      "i" is a QString, .toUtf8() returns a QByteArray, .constData() returns a
      pointer to the data inside the QByteArray.
      
      It is important to notice that the QByteArray is temporary. Therefore,
      it is "destroyed as the last step in evaluating the full-expression that
      (lexically) contains the point where [it was] created".
      
      Concretely, this means that this call is correct:
      
          do_something( qtu( string ) );
      
      But this one is undefined behavior:
      
          const char *s = qtu( string );
          do_something( s );
      
      Thus, here, bookmark.psz_name was initialized with a pointer to garbage
      data.
      
      To fix the problem, store the QByteArray in a local variable so that it
      lives long enough.
      
      (Fixes invalid reads reported by valgrind)
      Signed-off-by: Romain Vimont's avatarRomain Vimont <rom@rom1v.com>
      Signed-off-by: Jean-Baptiste Kempf's avatarJean-Baptiste Kempf <jb@videolan.org>
      ea0a32e1
  11. 30 Oct, 2017 1 commit
  12. 20 Oct, 2017 1 commit
  13. 14 Oct, 2017 4 commits
  14. 21 Sep, 2017 1 commit
  15. 15 Sep, 2017 1 commit
  16. 06 Sep, 2017 1 commit
  17. 05 Sep, 2017 1 commit
  18. 04 Sep, 2017 1 commit
  19. 22 Aug, 2017 2 commits
  20. 08 Aug, 2017 1 commit
  21. 23 Jul, 2017 4 commits
  22. 04 Jul, 2017 2 commits
  23. 31 May, 2017 1 commit
  24. 21 May, 2017 1 commit
  25. 18 May, 2017 1 commit
  26. 16 May, 2017 1 commit
  27. 14 May, 2017 1 commit
    • Rémi Denis-Courmont's avatar
      playlist: allocate the root item statically · 9c77c044
      Rémi Denis-Courmont authored
      This allocates space for the root playlist item directly inside the
      playlist structure. Creation of that item required a special case
      anyway as it was the only item without a parent.
      
      As a consequence, one unhandled error case is removed.
      9c77c044
  28. 25 Apr, 2017 1 commit
  29. 09 Apr, 2017 1 commit