mirror of
https://github.com/vanitasvitae/Smack.git
synced 2024-12-22 10:37:59 +01:00
[caps] Check for null in EntityCapsManager.addCapsExtension()
To avoid a NPE, we check for null in addCapsExtension. Fixes SMACK-899.
This commit is contained in:
parent
938a4271f3
commit
df3ca4100b
1 changed files with 6 additions and 3 deletions
|
@ -312,7 +312,10 @@ public final class EntityCapsManager extends Manager {
|
||||||
// XEP-0115 specifies that a client SHOULD include entity capabilities
|
// XEP-0115 specifies that a client SHOULD include entity capabilities
|
||||||
// with every presence notification it sends.
|
// with every presence notification it sends.
|
||||||
private void addCapsExtension(PresenceBuilder presenceBuilder) {
|
private void addCapsExtension(PresenceBuilder presenceBuilder) {
|
||||||
CapsVersionAndHash capsVersionAndHash = getCapsVersionAndHash();
|
final CapsVersionAndHash capsVersionAndHash = getCapsVersionAndHash();
|
||||||
|
if (capsVersionAndHash == null) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
CapsExtension caps = new CapsExtension(entityNode, capsVersionAndHash.version, capsVersionAndHash.hash);
|
CapsExtension caps = new CapsExtension(entityNode, capsVersionAndHash.version, capsVersionAndHash.hash);
|
||||||
presenceBuilder.overrideExtension(caps);
|
presenceBuilder.overrideExtension(caps);
|
||||||
}
|
}
|
||||||
|
@ -434,10 +437,10 @@ public final class EntityCapsManager extends Manager {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Get our own caps version. The version depends on the enabled features.
|
* Get our own caps version or {@code null} if none is yet set. The version depends on the enabled features.
|
||||||
* A caps version looks like '66/0NaeaBKkwk85efJTGmU47vXI='
|
* A caps version looks like '66/0NaeaBKkwk85efJTGmU47vXI='
|
||||||
*
|
*
|
||||||
* @return our own caps version
|
* @return our own caps version or {@code null}.
|
||||||
*/
|
*/
|
||||||
public CapsVersionAndHash getCapsVersionAndHash() {
|
public CapsVersionAndHash getCapsVersionAndHash() {
|
||||||
return currentCapsVersion;
|
return currentCapsVersion;
|
||||||
|
|
Loading…
Reference in a new issue