On Jan 28, 2011, at 6:14 PM, Samuel Ortiz <sa...@linux.intel.com> wrote: > On Fri, Jan 28, 2011 at 01:39:09PM -0800, Grant Erickson wrote: >> On Jan 28, 2011, at 12:10 PM, Grant Erickson wrote: >>> On Jan 28, 2011, at 10:09 AM, Grant Erickson wrote: >>>> On Jan 28, 2011, at 9:44 AM, Samuel Ortiz wrote: >>>>> On Fri, Jan 28, 2011 at 09:06:12AM -0800, Grant Erickson wrote: >>>>>> FYI. I need to check against GIT top-of-tree and dig into this further; >>>>>> however, with: >>>>>> >>>>>> * Wired Ethernet connected, connman-0.67 crashes on start-up >>>>>> * Wireless 802.11 WEXT, connman-0.67 hangs on start-up >>>>>> >>>>>> Backtrace with wired: >>>>> >>>>> Please give us a gdb backtrace or run test/backtrace src/connmand <log> >>>>> where >>>>> log is the below trace. >> >> The following patch fixes the core dump / fault issue, of which there >> were/are two core problems: >> >> 1) Trying to memory map the statistics file on armv7l-linux-unknown-gnu >> with MAP_SHARED results in a result value from mmap of MAP_FAILED. Using >> MAP_PRIVATE works >> correctly. This primary failure causes the secondary failure that >> leads to the segment fault. >> > Why does this toolchain fail to build a SHARED mapping?
That's the platform: Linux-2.6.32 > >> 2) The function stats_free, called indirectly from >> g_hash_table_remove(stats_hash, service), g_free(file->name) ends up >> double-freeing file->name. It had already been >> earlier freed in stats_file_setup, following the failure of >> status_file_remap. >> > This part of the patch makes sense, I'll apply it. > > Cheers, > Samuel. > > -- > Intel Open Source Technology Centre > http://oss.intel.com/ _______________________________________________ connman mailing list connman@connman.net http://lists.connman.net/listinfo/connman
