According to RFC3921bis: Subscription Attribute the subscription attribute is optional. Smack currently throws a NullPointException on receiving such a roster stanza.
Better handling of Roster error
I've added a simple Roster Push test (according to RFC3921bis-03: Roster Push) which reproduces this bug (provided by the RosterTest class within the test-unit directory).
Committed a patch to revision 11642.