cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Miguel Ferreira" <>
Subject Review Request 18114: Fix findbugs findings in cloudstack-service-console-proxy-rdpclient
Date Fri, 14 Feb 2014 11:28:18 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for cloudstack, daan Hoogland and Hugo Trippaers.

Repository: cloudstack-git


    Fix findbugs findings in cloudstack-service-console-proxy-rdpclient
    - 32 int shifted by an amount not in range -31..31:
        The shifts by 32 bits don't actually have any effect on the value
        (as shown by the tests)
    - possible null pointer dereference
    - repeated conditional test
    - field only ever set to null
        All other uses of the field were to check if it was null,
        which it was, so it was removed
    Other actions:
    - Upgrade jUnit to version 4
    - Add PowerMock dependency


  conditional PRE-CREATION 
  pom.xml 9dbd72b 
  services/console-proxy-rdp/rdpconsole/pom.xml 05585a1 
  services/console-proxy-rdp/rdpconsole/src/main/java/common/ 688693b 
  services/console-proxy-rdp/rdpconsole/src/main/java/streamer/ 3a718ba 
  services/console-proxy-rdp/rdpconsole/src/test/java/common/ PRE-CREATION

  services/console-proxy-rdp/rdpconsole/src/test/java/rdpclient/ 17bc4bd

  services/console-proxy-rdp/rdpconsole/src/test/java/streamer/ PRE-CREATION



Added tests for code that changed and tested in Devcloud


Miguel Ferreira

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message