Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f046e957 authored by Ted Bauer's avatar Ted Bauer
Browse files

Treat undefined flags as false in resource flag processor

Bug: 373685244
Test: m
Change-Id: Ifd813186d8b1998a2d5446c36a949a8825151d17
Flag: EXEMPT trivial change
parent da5885d5
Loading
Loading
Loading
Loading
+5 −7
Original line number Diff line number Diff line
@@ -25,7 +25,6 @@ import android.aconfig.nano.Aconfig.parsed_flags;
import android.annotation.NonNull;
import android.annotation.Nullable;
import android.content.res.Flags;
import android.content.res.XmlResourceParser;
import android.os.Environment;
import android.os.Process;
import android.util.ArrayMap;
@@ -247,13 +246,12 @@ public class AconfigFlags {
            negated = true;
            featureFlag = featureFlag.substring(1).strip();
        }
        final Boolean flagValue = getFlagValue(featureFlag);
        boolean shouldSkip = false;
        Boolean flagValue = getFlagValue(featureFlag);
        if (flagValue == null) {
            Slog.w(LOG_TAG, "Skipping element " + parser.getName()
                    + " due to unknown feature flag " + featureFlag);
            shouldSkip = true;
        } else if (flagValue == negated) {
            flagValue = false;
        }
        boolean shouldSkip = false;
        if (flagValue == negated) {
            // Skip if flag==false && attr=="flag" OR flag==true && attr=="!flag" (negated)
            Slog.i(LOG_TAG, "Skipping element " + parser.getName()
                    + " behind feature flag " + featureFlag + " = " + flagValue);