Fixed a bug introduced in r11787.

Jean-Noel Rouvignac

18.15.2015 1d4d8fbcbfdb06a56bfdb48dfeb883e1d74525f3

Fixed a bug introduced in r11787.

 opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/TargetAttr.java

@@ -40,32 +40,19 @@
 * A class representing an ACI's targetattr keyword.
 */
public class TargetAttr {
    /*
     * Enumeration representing the targetattr operator.
     */
    /** Enumeration representing the targetattr operator. */
    private EnumTargetOperator operator = EnumTargetOperator.EQUALITY;

    /*
     * Flags that is set if all user attributes pattern seen "*".
     */
    /** Flags that is set if all user attributes pattern seen "*". */
    private boolean allUserAttributes;

    /*
     * Flags that is set if all operational attributes pattern seen "+".
     */
    /** Flags that is set if all operational attributes pattern seen "+". */
    private boolean allOpAttributes;

    /*
     * HashSet of the attribute types parsed by the constructor.
     */
    /** Set of the attribute types parsed by the constructor. */
    private HashSet<AttributeType> attributes = new HashSet<AttributeType>();

    /**
     * HashSet of the operational attribute types parsed by the constructor.
     */
    /** Set of the operational attribute types parsed by the constructor. */
    private HashSet<AttributeType> opAttributes = new HashSet<AttributeType>();

    /*
    /**
     * Regular expression that matches one or more ATTR_NAME's separated by
     * the "||" token.
     */
@@ -170,7 +157,6 @@
        return allUserAttributes;
    }


    /**
     * This flag is set if the parsing code saw:
     * targetattr="+" or targetattr != "+".
@@ -231,16 +217,12 @@
     * TargetAttr's operator value applied to the test result.
     */

    public static boolean isApplicable(AttributeType a,
                                       TargetAttr targetAttr) {
        boolean ret;
    public static boolean isApplicable(AttributeType a, TargetAttr targetAttr) {
        if(targetAttr.isAllUserAttributes() && targetAttr.isAllOpAttributes()) {
            ret =
              !targetAttr.getOperator().equals(EnumTargetOperator.NOT_EQUALITY);
        } else
            ret=evalAttrType(a, targetAttr);

        return ret;
            return !targetAttr.getOperator().equals(EnumTargetOperator.NOT_EQUALITY);
        } else {
            return evalAttrType(a, targetAttr);
        }
    }

    /**
@@ -269,7 +251,8 @@
    private static boolean evalAttrType(AttributeType attrType, boolean allAttrs,
            HashSet<AttributeType> attrs, EnumTargetOperator op) {
        boolean ret = allAttrs || attrs.contains(attrType);
        if (op.equals(EnumTargetOperator.NOT_EQUALITY))
        if ((allAttrs || !attrs.isEmpty())
            && op.equals(EnumTargetOperator.NOT_EQUALITY))
        {
          return !ret;
        }

 opendj-server-legacy/src/test/java/org/opends/server/authorization/dseecompat/TargetAttrTestCase.java

@@ -407,7 +407,7 @@

  @Test(dataProvider = "targetAttrData")
  public void testTargetAttrStrings(String eqOperator, String targetAttrString,
    String attribute, Boolean expectedResult) throws Exception
    String attribute, boolean expectedResult) throws Exception
  {
    EnumTargetOperator op = EnumTargetOperator.createOperator(eqOperator);
    TargetAttr targetAttr = TargetAttr.decode(op, targetAttrString);
@@ -415,7 +415,7 @@
    if (attrType == null) {
      attrType = DirectoryServer.getDefaultAttributeType(attribute);
    }
    Boolean res = TargetAttr.isApplicable(attrType, targetAttr);
    boolean res = TargetAttr.isApplicable(attrType, targetAttr);
    Assert.assertEquals(res, expectedResult);
  }
}