libXvMC: Changes to 'master'

[GitLab Mirror]

 src/XvMC.c |   30 ++++++++++++++++++++----------
 1 file changed, 20 insertions(+), 10 deletions(-)

New commits:
commit a964f14b7b61bb8055d5144c96ecbba5dfae4a3e
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Sep 22 15:56:27 2018 -0700

    Fix sign comparison warnings for loop indexes
    
    XvMC.c: In function ‘XvMCListSurfaceTypes’:
    XvMC.c:124:19: warning: comparison between signed and unsigned integer
     expressions [-Wsign-compare]
          for(i = 0; i < rep.num; i++) {
                       ^
    XvMC.c: In function ‘XvMCListSubpictureTypes’:
    XvMC.c:185:26: warning: comparison between signed and unsigned integer
     expressions [-Wsign-compare]
                 for(i = 0; i < rep.num; i++) {
                              ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 28ad81420723175bfb163e20406af6d4d56bf4d4
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Sep 22 15:51:49 2018 -0700

    Need to check for -1, not 0, to determine if shmat() failed
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 68d0e5a122c6c76c19cc58ce9cea1424c7a5db11
Author: Neha Gupta <neha.g1 at samsung.com>
Date:   Fri Jun 5 08:20:48 2015 +0000

    Fix handling of shmKey in XvMCGetDRInfo
    
    If we store the result of shmget in a CARD32 (unsigned int), then
    checking if it returned -1 for an error by using >= 0 doesn't work.
    
    Also, once the request is flushed from the buffer (as XReply does),
    there's no guarantee the values in it are still valid, so it's better
    to rely on our local variable instead.
    
    Fixes: https://gitlab.freedesktop.org/xorg/lib/libxvmc/issues/1
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>