Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.python > #87138

Re: Question about importlib

From "Frank Millman" <frank@chagford.com>
Subject Re: Question about importlib
Date 2015-03-08 09:53 +0200
References <mdgttm$pm0$1@ger.gmane.org> <CAPTjJmrXp4MSO9f=xb_BRuPnRZ7XRKsKTkbFvo-e5N7Lr_MVVg@mail.gmail.com>
Newsgroups comp.lang.python
Message-ID <mailman.165.1425801224.21433.python-list@python.org> (permalink)

Show all headers | View raw


"Chris Angelico" <rosuav@gmail.com> wrote in message 
news:CAPTjJmrXp4MSO9f=xb_BRuPnRZ7XRKsKTkbFvo-e5N7Lr_MVVg@mail.gmail.com...
> On Sun, Mar 8, 2015 at 6:30 PM, Frank Millman <frank@chagford.com> wrote:
>> Actually, as I write this, I realise that there is a more important 
>> question
>> that had not occurred to me before. Is this a potential security risk? My
>> intention is that the caller would only call functions within my own
>> modules, but this could be used to call any arbitrary function.
>
> Here's an easy solution to both halves of your problem. It guarantees
> that arbitrary functions can't be called (or at least, that functions
> from arbitrary modules can't be called), and guarantees predictable
> performance:
>
> modules = {
>    "some_module": some_module,
>    "another_module": another_module,
> }
>
> module_name, func_name = func_name.rsplit('.', 1)
> module = modules.get(module_name)
> if module: getattr(module, func_name)(caller, xml_elem)
> else: cope with invalid choice of module
>
> You could programmatically populate the dictionary (eg from a list of
> acceptable module names) either with importlib or by pulling them from
> sys.modules. But whichever way you do it, you have an easy guarantee
> that arbitrary modules won't be imported, guaranteeing both security
> and performance in one stroke.

That is a neat solution. Thanks, Chris

Frank


Back to comp.lang.python | Previous | Next | Find similar | Unroll thread


Thread

Re: Question about importlib "Frank Millman" <frank@chagford.com> - 2015-03-08 09:53 +0200

csiph-web