Skip to content
  • Geddings Barrineau's avatar
    5aaf95d9
    Fixed copy paste bug in ActionUtils (#727) · 5aaf95d9
    Geddings Barrineau authored
    * Fixed IP_TOS match problem.
    
    In MatchUtils.java where the IP_TOS match string is converted into ECN and DSCP fields, incorrect values were passed into each fields constructor. To fix the problem, the ECN and DSCP bits had to be isolated and then passed into each constructor.
    
    * Fixed a bug that had to do with the order of bitwise operations when dealing with IP_TOS matches. By adding parenthesis, we ensure that the 'and' is done before the 'shift'.
    
    * Fixed bug that would cause a set_dst_port action to result in a set_src_port action instead. Simple copy paste error.
    5aaf95d9
    Fixed copy paste bug in ActionUtils (#727)
    Geddings Barrineau authored
    * Fixed IP_TOS match problem.
    
    In MatchUtils.java where the IP_TOS match string is converted into ECN and DSCP fields, incorrect values were passed into each fields constructor. To fix the problem, the ECN and DSCP bits had to be isolated and then passed into each constructor.
    
    * Fixed a bug that had to do with the order of bitwise operations when dealing with IP_TOS matches. By adding parenthesis, we ensure that the 'and' is done before the 'shift'.
    
    * Fixed bug that would cause a set_dst_port action to result in a set_src_port action instead. Simple copy paste error.
Loading