linux-kernel - Re: [PATCH] scripts/gdb: Fix loading of modules information

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <56FA2D52.2030402@siemens.com>
Date:	Tue, 29 Mar 2016 09:22:58 +0200
From:	Jan Kiszka <jan.kiszka@...mens.com>
To:	Daniel Wagner <wagi@...om.org>
Cc:	linux-kernel@...r.kernel.org,
	Daniel Wagner <daniel.wagner@...-carit.de>,
	Jiri Kosina <jkosina@...e.cz>,
	Rusty Russell <rusty@...tcorp.com.au>,
	Kieran Bingham <kieran.bingham@...aro.org>
Subject: Re: [PATCH] scripts/gdb: Fix loading of modules information

On 2016-03-29 09:13, Daniel Wagner wrote:
> On 03/29/2016 08:59 AM, Jan Kiszka wrote:
>> On 2016-03-29 08:41, Daniel Wagner wrote:
>>>          for module in module_list():
>>>              gdb.write("{address} {name:<19} {size:>8}  {ref}".format(
>>> -                address=str(module['module_core']).split()[0],
>>> +                module_layout=module['core_layout']
>>> +                addressr=str(module_layout['base'])
>>
>> This looks untested.
> 
> Yep, I send out the untested version. Need more coffee...
> 
>>>                  name=module['name'].string(),
>>>                  size=str(module['core_size']),
>>>                  ref=str(module['refcnt']['counter'])))
>>> diff --git a/scripts/gdb/linux/symbols.py b/scripts/gdb/linux/symbols.py
>>> index 627750c..8650ede 100644
>>> --- a/scripts/gdb/linux/symbols.py
>>> +++ b/scripts/gdb/linux/symbols.py
>>> @@ -108,7 +108,8 @@ lx-symbols command."""
>>>  
>>>      def load_module_symbols(self, module):
>>>          module_name = module['name'].string()
>>> -        module_addr = str(module['module_core']).split()[0]
>>> +        module_layout = module['core_layout']
>>> +        module_addr = str(module_layout['base'])
>>>  
>>>          module_file = self._get_module_file(module_name)
>>>          if not module_file and not self.module_files_updated:
>>>
>>
>> Thanks, but ad4db3b24a should already be on its way into stable.
> 
> Ah, I haven't got that fix yet because I am still on a older branch.
> Also my google-foo didn't help.
> 
> Sorry about that noise.

No problem. At least we now know another frequent user.

BTW, Kieran and some other people at Linaro started to think about how
to auto-test for such regressions - probably the only reliable way to
catch them.

Jan

-- 
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux