bta_jv_act: Avoid potential NULL dereference

When 'tc' is NULL (a condition we check for), 't' is never
set to a non-NULL value, yet is still dereferenced.  We move
our NULL check of 't' to make this code safe.

Test: TreeHugger
Change-Id: Ibfea74963dd763401ea18ccdd24a51ecd0baefc5

parent baaeb80c

system/bta/jv/bta_jv_act.cc

+5 −4

Original line number	Diff line number	Diff line
		@@ -2117,11 +2117,12 @@ static void fcchan_data_cbk(uint16_t chan, const RawAddress& bd_addr,
		if (tc) {
		// try to find an open socked for that addr and channel
		t = fcclient_find_by_addr(tc->clients, &bd_addr);
		}
		if (!t) {
		// no socket -> drop it
		return;
		}
		}


		sock_cback = t->p_cback;
		sock_id = t->l2cap_socket_id;