gatppp: fix ppp protocol-reject constructing error

In PPP Protocol-Reject package the 'rejected data field' should be a copy
of the rejected package, the copy starts from the data field. besides,
Protocol-Reject package include a 'rejected protocol field', it is the
same as the protocol field of the rejected package.

Protocol-Reject package structure is:

| PPP_header | rejected protocol | rejected data

The rejected package structure is:
| Addr | Control | protocol | data

So the Protocol-Reject package data field is copied from the 3rd byte of
the rejected package.

simfs: Fix error reported by valgrind

==20365== Invalid read of size 8
==20365==    at 0x4B3501: sim_fs_free (simfs.c:114)
==20365==    by 0x493CEC: sim_remove (sim.c:2485)
==20365==    by 0x4703D7: modem_change_state (modem.c:410)
==20365==    by 0x470664: set_powered (modem.c:848)
==20365==    by 0x4706BA: __ofono_modem_shutdown (modem.c:2137)
==20365==    by 0x46F2C5: signal_cb (main.c:76)
==20365==    by 0x52F555E: g_io_unix_dispatch (giounix.c:166)
==20365==    by 0x52A0AAB: g_main_dispatch (gmain.c:2440)
==20365==    by 0x52A203B: g_main_context_dispatch (gmain.c:3013)
==20365==    by 0x52A2501: g_main_context_iterate (gmain.c:3091)
==20365==    by 0x52A2C98: g_main_loop_run (gmain.c:3299)
==20365==    by 0x46F0D3: main (main.c:243)
==20365==  Address 0x63ff998 is 8 bytes inside a block of size 16 free'd
==20365==    at 0x4C2612D: free (in
/usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==20365==    by 0x52AA4A1: g_free (gmem.c:263)
==20365==    by 0x52C433D: g_slice_free1 (gslice.c:907)
==20365==    by 0x52C58D0: g_slist_free_1 (gslist.c:192)
==20365==    by 0x52C5C5F: g_slist_remove (gslist.c:465)
==20365==    by 0x4B342A: sim_fs_context_free (simfs.c:192)
==20365==    by 0x4B3500: sim_fs_free (simfs.c:117)
==20365==    by 0x493CEC: sim_remove (sim.c:2485)
==20365==    by 0x4703D7: modem_change_state (modem.c:410)
==20365==    by 0x470664: set_powered (modem.c:848)
==20365==    by 0x4706BA: __ofono_modem_shutdown (modem.c:2137)
==20365==    by 0x46F2C5: signal_cb (main.c:76)

simfs: Fix issue reported by valgrind

==20287== Invalid read of size 4
==20287==    at 0x52B5C6B: g_queue_peek_nth_link (gqueue.c:704)
==20287==    by 0x52B5F57: g_queue_peek_nth (gqueue.c:848)
==20287==    by 0x4B33ED: sim_fs_context_free (simfs.c:170)
==20287==    by 0x4B34F8: sim_fs_free (simfs.c:116)
==20287==    by 0x493CEC: sim_remove (sim.c:2485)
==20287==    by 0x4703D7: modem_change_state (modem.c:410)
==20287==    by 0x470664: set_powered (modem.c:848)
==20287==    by 0x4706BA: __ofono_modem_shutdown (modem.c:2137)
==20287==    by 0x46F2C5: signal_cb (main.c:76)
==20287==    by 0x52F555E: g_io_unix_dispatch (giounix.c:166)
==20287==    by 0x52A0AAB: g_main_dispatch (gmain.c:2440)
==20287==    by 0x52A203B: g_main_context_dispatch (gmain.c:3013)
==20287==  Address 0x63fae70 is 16 bytes inside a block of size 24
free'd
==20287==    at 0x4C2612D: free (in
/usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==20287==    by 0x52AA4A1: g_free (gmem.c:263)
==20287==    by 0x52C433D: g_slice_free1 (gslice.c:907)
==20287==    by 0x52B4E96: g_queue_free (gqueue.c:60)
==20287==    by 0x4B34E0: sim_fs_free (simfs.c:107)
==20287==    by 0x493CEC: sim_remove (sim.c:2485)
==20287==    by 0x4703D7: modem_change_state (modem.c:410)
==20287==    by 0x470664: set_powered (modem.c:848)
==20287==    by 0x4706BA: __ofono_modem_shutdown (modem.c:2137)
==20287==    by 0x46F2C5: signal_cb (main.c:76)
==20287==    by 0x52F555E: g_io_unix_dispatch (giounix.c:166)
==20287==    by 0x52A0AAB: g_main_dispatch (gmain.c:2440)

gprs: Mark context inactive on removal

gprs: Fix valgrind error

This fixes an invalid access condition for the case where the
gprs atom is destroyed before context atoms are.

gprs: emit Active changed for contexts being removed

gprs: Add additional debug statements

phonesim: Create atoms in a different order

This uncovers a valgrind condition:

==17231== Invalid read of size 8
==17231==    at 0x4A2461: gprs_context_unregister (gprs.c:2143)
==17231==    by 0x46FEB0: __ofono_atom_unregister (modem.c:260)
==17231==    by 0x4703C9: modem_change_state (modem.c:407)
==17231==    by 0x470664: set_powered (modem.c:848)
==17231==    by 0x4706BA: __ofono_modem_shutdown (modem.c:2137)

stkutil: Send at least the data currently available

sim: Fix potential use of uninitialized variable

In certain circumstances, when the image has been cached but EFimg has
not been read yet, we might end up accessing an unitialized variable.
Fix this by always failing if EFimg has not been read yet.

emulator: add AT+CMEE support for HFP

emulator: add dun_ath_cb() and register it

emulator: add handler for ppp stream suspensions

Resulting from the client sending '+++' escape sequence.

gatppp: Add API for setting suspend callback

In situations where the PPP stream can be suspended by the peer, e.g. in
the server role using '+++'

ppp_net: Reset the watch to zero when removing

ppp_net: add ppp_net_suspend_interface() function

gathdlc: delete read/write handler in hdlc_suspend

gathdlc: Add public suspend function

gatchat: Fix memory leak with notify nodes list

The list itself needs to be freed as well and not only its elements.

call-forwarding: Fix uninitialized var warning

Fix a gcc 4.5.2 warning uninit var

voicecall: CHLD support must be enclosed in a list

voicecall: Simplify CHLD support generator

In addition, fix ordering and make sure that lower case 'x' is used

voicecall: CHLD call ids start at 1, not 0

voicecall: add +CHLD=2X support for HFP emulator

voicecall: add +CHLD=1X support for HFP emulator

voicecall: add +CHLD=4 support for HFP emulator

voicecall: add +CHLD=3 support for HFP emulator

voicecall: add +CHLD=0 support for HFP emulator

voicecall: add +CHLD support for HFP emulator

voicecall: Fix variable initialization

gathdlc: Make sure to set in_read_handler properly

gathdlc: Add support for detecting +++ escapes

gatio: add ability to drain GAtIO read buffer

gat: add GAtSuspendFunc CB typedef

sms: Remove use of uninitialized variable

Remove a debug information, since the info is useless, and more,
it can display invalid info due to var not initialise in some case

modem: Don't allow setting online unless powered

test: Add swap-calls test script

gisi: log normal responses in a less confusing way

build: Add send-vcal to Makefile.am

test: Add send-vcal test script

test: Add modem argument to send-vcard

test: Add modem argument to send-sms

Release 0.48

huawei: Switch to offline mode only if offline

huawei: Check that the modem device is open

Don't create the gprs atom if the modem device was not opened
successfully.  Otherwise ofonod will crash.

gatppp: Fix crash related to not stopping timers

In case of offline modem when GPRS data connection is connected, if gprs
atom is removed before PPP termination process is complete, the terminate_timer
will not be stop. It will cause ofonod crash when the timer times out.

voicecall: fix waiting calls for HFP emulator

Fix precedence between waiting and dialing so HFP emulator is notified
of dialing/alerting even in case of a waiting call pending.

voicecall: Update CLCC buffer size and comment

voicecall: Refactor the previous commit slightly

voicecall: add +CLCC support for HFP emulator

emulator: fix +CLIP in case of withheld call

emulator: Create both emulator types in example

hfp_ag: Be extra pedantic on plugin cleanup

dun_gw: Be extra pedantic on plugin cleanup

dun_gw: Minor style tweak

hfp_ag: Minor style tweak

voicecall: Tweak the behavior of previous commit

Use a generic void pointer to remove the need for emulator_release_done
function.

Use naming a bit more consistent with the rest of the codebase

voicecall: add +CHUP support for HFP emulator

Update multirelease_callback to be used from DBus calls or HFP Emulator.
"release done" is configurable by multirelease caller.

voicecall: Don't go over 80 characters per line

voicecall: add ATA support for HFP emulator

emulator: Use a switch/case instead of if

Seems a bit cleaner

emulator: Reflow comments

emulator: add +CCWA support for HFP AG

emulator: add +CLIP support for HFP AG

voicecall: add API to find call by status

emulator: add RING for HFP AG

isiusb: wait for MTC transitions, too

ussd: Fix use of non-cloned gatchat object

Release 0.47

TODO: Remove main.conf task

No longer seems like a good idea

TODO: set owner on DUN profile and DUN usb

TODO: Mark EFcsp task as done

simutil: Add support of CPHS EFcsp to our EF db

network: One more trivial fix

total_length and record_length are the same in this context, but using
total_length makes the code more readable

doc: Update possible errors for netreg-api

This is due to EFcsp PLMN mode changes

network: Refactor EFcsp to support STK Refresh

network: Use strings instead of ints in settings

modem: Fix entering offline mode prematurely

A contextual change to __ofono_modem_find_atom has caused us to enter
into offline mode prematurely.  find_atom was changed to return only the
atoms that were registered, however the modem really has to know whether
the sim atom is created (not registered) in order to know whether to
wait for the SIM to be initialized or not.

sms: Fix typo

network: Read EFcsp after initializing reg state

This is to avoid some benign race conditions when EFcsp might be read
before the registration state has been established.

network: Try not to clash with 27.007-defined modes

netreg: Refactor register_callback

netreg: Add CPHS CSP implementation

isimodem: Add SCA support to wgmodem2.5 driver

isimodem: Refactor sms driver

isimodem: Use constants where possible

isimodem: Fix message sending in SMS driver

The same enum name SMS_ROUTE_DEFAULT has different constants in
different modem API versions. This was causing "invalid parameter"
errors with some modems.

isimodem: Fix macro to be safer to use

isimodem: Add reahcbility check for CBS driver

doc: add emergency-call-handling.txt

Release 0.46

isimodem: fix call_create_resp handling

In case of call creation failure modem may return a valid call id in
order to send CALL_SERVICE_DENIED_IND which we do not handle.

Fixes MeeGo bug#15855.

isimodem: Update SMS driver for wgmodem2.5

Based on patches from:
Jessica Nilsson <jessica.j.nilsson@stericsson.com>

plugins: Remove wgmodem2.5 driver for netreg

isimodem: Remove split voicecall driver

Only a single driver created, with two clients checked for
reachability in parallel.

isimodem: Add helper macros for 16/32bit values

isimodem: Rename resource debug macro

isimodem: Add helper macro for version checking

isimodem: Indent and add missing codepoints

In addition, renaming some wgmodem2.5-specific subblocks so that they
use the same names as most recent on-line documentation.

doc: Mark ConfirmOpenChannel as experimental