>> @@ -311,6 +315,10 @@ void telephony_transmit_dtmf_req(void *telephony_device, char tone)
>>
>> debug("telephony-ofono: transmit dtmf: %c", tone);
>>
>> + if (!modem_obj_path)
>> + telephony_transmit_dtmf_rsp(telephony_device,
>> + CME_ERROR_AG_FAILURE);
>> +
>> tone_string = g_strdup_printf("%c", tone);
>> ret = send_method_call(OFONO_BUS_NAME, modem_obj_path,
>
> This doesn't look right. You report an error but still proceed in the
> functions (which will later try to access modem_obj_path). I guess you
> should return from the functions in both of these if-clauses.
>
My bad. Will send out the updated patch ASAP.
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
