October 2012 - ofono - Ml01.01.Org

[PATCH_v4 0/5] Private network request to ConnMan

by Guillaume Zajac

Hi, Changelog from v3 is: - Add private-network source/include - ConnMan plugin is independant from emulator - Each application that need VPN will pass a callback as argument when private network is requested. This callback will contain the private network settings. Guillaume Zajac (5): gatppp: Add new contructor to use external fd private-network: add callback typedef drivers and settings private-network: add request/release functions and new feature to Makefile.am emulator: add request/release private network calls connman: add plugin in oFono to request request/release private network Makefile.am | 10 +- gatchat/gatppp.c | 33 +++++- gatchat/gatppp.h | 1 + gatchat/ppp.h | 2 +- gatchat/ppp_net.c | 40 ++++--- include/private-network.h | 59 +++++++++ plugins/connman.c | 297 +++++++++++++++++++++++++++++++++++++++++++++ src/emulator.c | 49 ++++++-- src/ofono.h | 6 + src/private-network.c | 89 ++++++++++++++ 10 files changed, 556 insertions(+), 30 deletions(-) create mode 100644 include/private-network.h create mode 100644 plugins/connman.c create mode 100644 src/private-network.c

2 months, 1 week

5
12
0 / 0

[PATCH 1/4] gdbus: Fix crash when getting disconnected from the bus

by Luiz Augusto von Dentz

From: Johan Hedberg <johan.hedberg(a)intel.com> When getting disconnected from the bus sometimes (maybe always?) dbus_watch_handle() can cause the "info" context to be free'd meaning that we should not try to access it after the call. The only member we need access to is the connection pointer and as the code already has a ref() call for it it's only natural to solve the issue by adding a local variable not dependent on "info". The backtrace of the crash fixed looks as follows: Invalid read of size 8 at 0x121085: watch_func (mainloop.c:105) by 0x4C72694: g_main_context_dispatch (gmain.c:2539) by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146) by 0x4C72DC1: g_main_loop_run (gmain.c:3340) by 0x120541: main (main.c:551) Address 0x5bbcd90 is 16 bytes inside a block of size 24 free'd at 0x4A079AE: free (vg_replace_malloc.c:427) by 0x4C7837E: g_free (gmem.c:252) by 0x4F708BF: dbus_watch_set_data (dbus-watch.c:614) by 0x4F70938: _dbus_watch_unref (dbus-watch.c:132) by 0x4F6E9A7: _dbus_transport_handle_watch (dbus-transport.c:884) by 0x4F59AFB: _dbus_connection_handle_watch (dbus-connection.c:1497) by 0x4F70AF9: dbus_watch_handle (dbus-watch.c:683) by 0x121084: watch_func (mainloop.c:103) by 0x4C72694: g_main_context_dispatch (gmain.c:2539) by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146) by 0x4C72DC1: g_main_loop_run (gmain.c:3340) by 0x120541: main (main.c:551) --- gdbus/mainloop.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/gdbus/mainloop.c b/gdbus/mainloop.c index cff326f..099b67f 100644 --- a/gdbus/mainloop.c +++ b/gdbus/mainloop.c @@ -92,8 +92,9 @@ static gboolean watch_func(GIOChannel *chan, GIOCondition cond, gpointer data) struct watch_info *info = data; unsigned int flags = 0; DBusDispatchStatus status; + DBusConnection *conn; - dbus_connection_ref(info->conn); + conn = dbus_connection_ref(info->conn); if (cond & G_IO_IN) flags |= DBUS_WATCH_READABLE; if (cond & G_IO_OUT) flags |= DBUS_WATCH_WRITABLE; @@ -102,10 +103,10 @@ static gboolean watch_func(GIOChannel *chan, GIOCondition cond, gpointer data) dbus_watch_handle(info->watch, flags); - status = dbus_connection_get_dispatch_status(info->conn); - queue_dispatch(info->conn, status); + status = dbus_connection_get_dispatch_status(conn); + queue_dispatch(conn, status); - dbus_connection_unref(info->conn); + dbus_connection_unref(conn); return TRUE; } -- 1.7.11.4

8 years, 10 months

2
4
0 / 0

[PATCH] sim: Fix the quirk handling of CPIN for Wavecom Q2XXX

by Holger Hans Peter Freyther

From: Holger Hans Peter Freyther <hfreyther(a)sysmocom.de> Commit 1ccecfe01eb7f9f4d1d931814deb66ef37840b73 added the quirk handling for the Q2XXX module but it missed the first part of the Wavecom quirk handling. If decode_at_error is not fed with OK the error variable will be set to an error and the sim_pin_query_cb will exit before doing any notifications. This started to happen after git commit 72ce19bf3e87b5e73c053f7ea1095a5006f1aea9). --- drivers/atmodem/sim.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/atmodem/sim.c b/drivers/atmodem/sim.c index cf3345c..89445bb 100644 --- a/drivers/atmodem/sim.c +++ b/drivers/atmodem/sim.c @@ -980,7 +980,8 @@ static void at_cpin_cb(gboolean ok, GAtResult *result, gpointer user_data) int len = sizeof(at_sim_name) / sizeof(*at_sim_name); const char *final = g_at_result_final_response(result); - if (sd->vendor == OFONO_VENDOR_WAVECOM && ok && strlen(final) > 7) + if ((sd->vendor == OFONO_VENDOR_WAVECOM || + sd->vendor == OFONO_VENDOR_WAVECOM_Q2XXX) && ok && strlen(final) > 7) decode_at_error(&error, "OK"); else decode_at_error(&error, final); -- 1.7.10.4

8 years, 11 months

2
1
0 / 0

Ofono can't work with ZTE MF190S

by Yevhen Kyriukha

Hello! I have a problems with Ofono while working with my 3G modem *ZTE MF190S*. I can't connect to GPRS network. I'm getting an error when I run *enable-gprs* test script. Log files are attached. As you can see there are messages like *GPRS not supported on this device* and *Unable to initialize Network Registration* in ofono.log and ofono-at.log. -- Best regards, Yevhen

8 years, 11 months

2
1
0 / 0

[PATCH 1/1] Set MNC length to 2 if EFad doesn't specify it

by Jonas Bonn

Byte 4 of the EFAD structure, which specifies the MNC length, is optional. If this byte is not present, then there is no way to determine the length of the MNC. As the, by far, most common length is 2, we might as well default to this value. This patch changes the ofono_error to a warning as it's not really an error that we're encountering, but the user should nonetheless be given a heads-up that something might not work as expected. Signed-off-by: Jonas Bonn <jonas(a)southpole.se> --- src/sim.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/sim.c b/src/sim.c index 4384eb0..f6109c6 100644 --- a/src/sim.c +++ b/src/sim.c @@ -1760,7 +1760,9 @@ static void sim_ad_read_cb(int ok, int length, int record, return; if (length < 4) { - ofono_error("EFad should contain at least four bytes"); + ofono_warn("EFad does not specify (optional) MNC length" + ", defaulting to 2"); + sim->mnc_length = 2; return; } -- 1.7.9.5

8 years, 11 months

2
1
0 / 0

Ofono code flow info

by Rohit Pathania

Hi Sir/Mam, I want to get information on ofono and its dbus api. How to build it. So I can write some application by using ofono. I am doing this because of interste in telephony. so please help in it. Thanks & Regards, Rohit

8 years, 11 months

1
0
0 / 0

[PATCH] systemd: prevent duplicate logging messages in journal

by Marti Raudsepp

By default, both stderr and syslog messages go to the systemd journal, which results in duplicate messages being logged. Thanks to Vinicius Costa Gomes for pointing out this problem. --- dundee/dundee.service.in | 1 + src/ofono.service.in | 1 + 2 files changed, 2 insertions(+) diff --git a/dundee/dundee.service.in b/dundee/dundee.service.in index c57c618..82c5ef1 100644 --- a/dundee/dundee.service.in +++ b/dundee/dundee.service.in @@ -6,6 +6,7 @@ After=syslog.target Type=dbus BusName=org.ofono.dundee ExecStart=@prefix@/sbin/dundee -n +StandardError=null [Install] WantedBy=multi-user.target diff --git a/src/ofono.service.in b/src/ofono.service.in index f4531a8..c24ac28 100644 --- a/src/ofono.service.in +++ b/src/ofono.service.in @@ -6,6 +6,7 @@ After=syslog.target Type=dbus BusName=org.ofono ExecStart=@prefix@/sbin/ofonod -n +StandardError=null [Install] WantedBy=multi-user.target -- 1.8.0

8 years, 11 months

2
1
0 / 0

oFono support for the Telit GC864-QUAD V2 serial port GSM/GPRS modem

by Markus Hubig

Hello @all, I'm new here and I'm in the process of developing a Embedded Linux datalogger appliance. At the moment I'm evaluating the modem part of it, and we are looking forward to use the Telit GC864-QUAD V2 serial port GSM/GPRS modem with oFono. Unfortunately I couldn't find much "getting started" information for oFono, but by reading throughout the old posts on this list I gathered at least some. Please correct me if I get things wrong: 1. The one-and-only way to tell oFono to use a modem is by udev rules like this: KERNEL=="ttyUSB0", ENV{OFONO_DRIVER}="calypso" 2. Serial port GPRS modems like the Telit GC864 are not supported out-of-the-box, so one has to write a special modem driver for it. 3. Since this modem supports CMUX multiplexing with the 3GPP 27.010 standard, I can use 'GAtMux' like it's done in 'plugins/calypso.c'. 4. There is a telit modem driver, but this is only for the UC864 UMTS modem which is connected via USB. So in order to write a driver to support the GC864, what would be a good starting point? Add support for serial port CMUX to the telit driver? Looking forward for some advice ... Cheers, Markus

8 years, 11 months

2
3
0 / 0

Fail to register to default network - maybe a race condition

by Martin Grimme

Hi, I'm experiencing a strange behavior with ofono (1.10) and the Huawei EM770W modem on Pegatron Lucid (WeTab) hardware. The same used to work well with ofono 1.8. As soon as I get notified about the availability of org.ofono.NetworkRegistration, I'm calling org.ofono.NetworkRegistration.Register on that modem. However, this shows no effect. See http://pastebin.com/CAVEvsA2 for a trace of ofonod. But when using the test-network-registration test script from ofono to connect to the default network, it works, even though the script also only uses org.ofono.NetworkRegistration.Register to register. See http://pastebin.com/9RQbd8QU for a trace of ofonod. The code I'm using for registration is here: http://pastebin.com/fkVVVc1a Regards, Martin

8 years, 11 months

1
0
0 / 0

[PATCH] smsutil: Remove compilation warning

by Henrique Dante de Almeida

--- src/smsutil.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/smsutil.c b/src/smsutil.c index e3b4b72..7399fbe 100644 --- a/src/smsutil.c +++ b/src/smsutil.c @@ -4108,7 +4108,7 @@ char *cbs_decode_text(GSList *cbs_list, char *iso639_lang) */ for (; i < written; i++, bufsize++) { if (unpacked[i] == '\r') { - unsigned int t; + long t; t = strspn((const char *) unpacked + i, "\r"); -- 1.7.12.rc2.16.g034161a

8 years, 11 months

1
0
0 / 0

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

ofono October 2012