From 659714c2be402de0e3a02a90bec2238281ede671 Mon Sep 17 00:00:00 2001 From: Luke GB Date: Sat, 23 Aug 2014 17:24:53 +0100 Subject: [PATCH] Set jid to passed parameter, not null Setting `jid` to `null` was probably not the intended behaviour of this constructor and means that the `IQReplyFilter` will later reject responses that it shouldn't, because the `AbstractXmppConnection` will automatically construct a JID based on the authenticated username. In particular, this breaks attempting to connect to the PVP.net XMPP server. --- .../src/main/java/org/jivesoftware/smack/packet/Bind.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/smack-core/src/main/java/org/jivesoftware/smack/packet/Bind.java b/smack-core/src/main/java/org/jivesoftware/smack/packet/Bind.java index df424ed75..7a5bada19 100644 --- a/smack-core/src/main/java/org/jivesoftware/smack/packet/Bind.java +++ b/smack-core/src/main/java/org/jivesoftware/smack/packet/Bind.java @@ -40,7 +40,7 @@ public class Bind extends IQ { public Bind(String resource, String jid) { this.resource = resource; - this.jid = null; + this.jid = jid; } public String getResource() {