From: John Ernberg <[email protected]>

Casting between incompatible function pointer types is undefined.
While it works fine on x86, it's not a good idea to depend on this.

This RFC uses g_slist_free_full where possible, and when it's not works around
the casting using other means such as wrappers or fixing function parameters.

John Ernberg (24):
  atmodem: don't rely on undefined behavior when casting function
    pointers
  hfpmodem: don't rely on undefined behavior when casting function
    pointers
  ifxmodem: don't rely on undefined behavior when casting function
    pointers
  ril: don't rely on undefined behavior when casting function pointers
  stemodem: don't rely on undefined behavior when casting function
    pointers
  gatchat: don't rely on undefined behavior when casting function
    pointers
  bluez4: don't rely on undefined behavior when casting function
    pointers
  sm: don't rely on undefined behavior when casting function pointers
  cbs: don't rely on undefined behavior when casting function pointers
  cdma/sms: don't rely on undefined behavior when casting function
    pointers
  handsfree: don't rely on undefined behavior when casting function
    pointers
  modem: don't rely on undefined behavior when casting function pointers
  network: don't rely on undefined behavior when casting function
    pointers
  phonebook: don't rely on undefined behavior when casting function
    pointers
  sim: don't rely on undefined behavior when casting function pointers
  simfs: don't rely on undefined behavior when casting function pointers
  simutil: don't rely on undefined behavior when casting function
    pointers
  sms: don't rely on undefined behavior when casting function pointers
  smsutil: don't rely on undefined behavior when casting function
    pointers
  stk: don't rely on undefined behavior when casting function pointers
  stkutil: don't rely on undefined behavior when casting function
    pointers
  ussd: don't rely on undefined behavior when casting function pointers
  voicecall: don't rely on undefined behavior when casting function
    pointers
  unittest: don't rely on undefined behavior when casting function
    pointers

 drivers/atmodem/voicecall.c  |  6 ++----
 drivers/hfpmodem/voicecall.c |  6 ++----
 drivers/ifxmodem/voicecall.c |  3 +--
 drivers/rilmodem/voicecall.c |  6 ++----
 drivers/stemodem/voicecall.c |  3 +--
 gatchat/gatchat.c            | 13 +++++--------
 plugins/bluez4.c             |  7 +++----
 plugins/smart-messaging.c    |  6 ++----
 src/cbs.c                    | 27 +++++++++------------------
 src/cdma-smsutil.c           |  3 +--
 src/handsfree.c              |  3 +--
 src/modem.c                  |  6 ++----
 src/network.c                |  3 +--
 src/phonebook.c              | 24 ++++++++++++++----------
 src/sim.c                    | 29 ++++++++++-------------------
 src/simfs.c                  |  6 +++---
 src/simutil.c                |  6 ++----
 src/sms.c                    |  9 +++------
 src/smsutil.c                | 21 +++++++--------------
 src/stk.c                    | 10 +++++++---
 src/stkutil.c                | 39 ++++++++++++---------------------------
 src/ussd.c                   | 10 +++++-----
 src/voicecall.c              |  7 ++-----
 unit/test-sms.c              | 21 +++++++--------------
 24 files changed, 104 insertions(+), 170 deletions(-)

-- 
1.9.1
_______________________________________________
ofono mailing list
[email protected]
https://lists.ofono.org/mailman/listinfo/ofono

Reply via email to