sling-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dan Klco (Jira)" <j...@apache.org>
Subject [jira] [Resolved] (SLING-8858) Model Cache causes NPE in sling.models.impl 1.4.10+
Date Mon, 16 Mar 2020 12:35:00 GMT

     [ https://issues.apache.org/jira/browse/SLING-8858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Dan Klco resolved SLING-8858.
-----------------------------
    Resolution: Fixed

Fixed inĀ [https://github.com/apache/sling-org-apache-sling-models-impl/pull/18/commits/92c0a7289924f209332af675311c044338347b62]

> Model Cache causes NPE in sling.models.impl 1.4.10+
> ---------------------------------------------------
>
>                 Key: SLING-8858
>                 URL: https://issues.apache.org/jira/browse/SLING-8858
>             Project: Sling
>          Issue Type: Bug
>          Components: Sling Models
>    Affects Versions: Sling Models Impl 1.4.10
>         Environment: Java SE 1.8.0_231
>            Reporter: Frank Pauleickhoff
>            Assignee: Dan Klco
>            Priority: Major
>             Fix For: Sling Models Impl 1.4.14
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> In AEM 6.4 ServicePack 6 we face NullPointerExceptions when using the Sling model cache
that used to work before (in SP 3).
> *Exception Stacktrace is:*
> {code:java}
> 22.11.2019 15:29:22.525 *ERROR* [127.0.0.1 [1574432957337] GET /content/mypage.html HTTP/1.1]
org.apache.sling.engine.impl.SlingRequestProcessorImpl service: Uncaught SlingException
> org.apache.sling.scripting.sightly.SightlyException: org.apache.sling.api.SlingException:
Cannot get DefaultSlingScript: org.apache.sling.api.SlingException: Cannot get DefaultSlingScript:
org.apache.sling.api.SlingException: Cannot get DefaultSlingScript: org.apache.sling.api.SlingException:
Cannot get DefaultSlingScript: org.apache.sling.api.SlingException: Cannot get DefaultSlingScript:
org.apache.sling.api.SlingException: Cannot get DefaultSlingScript: Identifier xx.xxx.MyModel
cannot be correctly instantiated by the Use API
> [...]
> Caused by: java.lang.NullPointerException: null
>  at org.apache.sling.models.impl.ModelAdapterFactory.internalCreateModel(ModelAdapterFactory.java:407)
>  at org.apache.sling.models.impl.ModelAdapterFactory.createModel(ModelAdapterFactory.java:314)
> {code}
> Problem seems to be related to introduction of WeakReferences in SLING-7586. Without
knowing any details and being deeply into the Sling Model Caching implementation, I think
a null-check is missing for "SoftReference" in Line 407:
> {code:java}
> SoftReference<Object> SoftReference = adaptableCache.get(requestedType);
> ModelType cachedObject = (ModelType) SoftReference.get();
> {code}
> That seems to cause caching to throw NPE for any not yet cached model.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message