/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.karaf.jaas.config.impl;

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.concurrent.CopyOnWriteArrayList;

import javax.security.auth.login.AppConfigurationEntry;
import javax.security.auth.login.Configuration;

import org.apache.karaf.jaas.config.JaasRealm;

public class OsgiConfiguration extends Configuration {

    private final List<JaasRealm> realms = new CopyOnWriteArrayList<JaasRealm>();

    public void init() {
        Configuration.setConfiguration(this);
    }

    public void close() {
        realms.clear();
        Configuration.setConfiguration(null);
    }

    public void register(JaasRealm realm, Map<String,?> properties) {
        realms.add(realm);
    }

    public void unregister(JaasRealm realm, Map<String,?> properties) {
        realms.remove(realm);
    }

    public AppConfigurationEntry[] getAppConfigurationEntry(String name) {
        JaasRealm realm = null;
        for (JaasRealm r : realms) {
            if (r.getName().equals(name)) {
                if (realm == null || r.getRank() > realm.getRank()) {
                    realm = r;
                }
            }
        }
        if (realm != null) {
            return realm.getEntries();
        }
        return null;
    }

    public void refresh() {
        // Nothing to do, as we auto-update the configuration
    }
}

@@ -844,63 +844,8 @@
 l;%0A%0A
-import java.util.ArrayList;%0Aimport java.util.Iterator;%0A
 impo
@@ -935,17 +935,16 @@
 ayList;%0A
-%0A
 import j
@@ -1470,32 +1470,65 @@
 %3E properties) %7B%0A
+        if (realm != null) %7B%0A    
         realms.a
@@ -1530,32 +1530,42 @@
 lms.add(realm);%0A
+        %7D%0A
     %7D%0A%0A    publi
@@ -1619,32 +1619,65 @@
 %3E properties) %7B%0A
+        if (realm != null) %7B%0A    
         realms.r
@@ -1686,24 +1686,34 @@
 ove(realm);%0A
+        %7D%0A
     %7D%0A%0A    p

Fix NPE thrown from OsgiConfiguration.unregister