diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-01-27 23:41:22 (GMT) |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-01-28 12:44:14 (GMT) |
commit | 6caa7d1d6c122d084ed01d1f5e1dae0bf259f854 (patch) | |
tree | 7f2f7bd4d983ecfedd4154d6cecdae5f5a40c396 | |
parent | 2a69358610077d134cef38d37f24eddfc73fc294 (diff) | |
download | poky-denzil.tar.gz |
bitbake: command: Fix getCmdLineAction bugsdenzil
Executing "bitbake" doesn't get a sane message since the None return value
wasn't being handled correctly. Also fix msg -> cmd_action['msg'] as
otherwise an invalid variable is accessed which then crashes the server
due to the previous bug.
(Bitbake rev: c6211291ae07410832031a5274690437cc2b09a6)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | bitbake/lib/bb/command.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/bitbake/lib/bb/command.py b/bitbake/lib/bb/command.py index c08e2ce..a143aed 100644 --- a/bitbake/lib/bb/command.py +++ b/bitbake/lib/bb/command.py | |||
@@ -148,8 +148,10 @@ class CommandsSync: | |||
148 | Get any command parsed from the commandline | 148 | Get any command parsed from the commandline |
149 | """ | 149 | """ |
150 | cmd_action = command.cooker.commandlineAction | 150 | cmd_action = command.cooker.commandlineAction |
151 | if cmd_action['msg']: | 151 | if cmd_action is None: |
152 | raise CommandError(msg) | 152 | return None |
153 | elif 'msg' in cmd_action and cmd_action['msg']: | ||
154 | raise CommandError(cmd_action['msg']) | ||
153 | else: | 155 | else: |
154 | return cmd_action['action'] | 156 | return cmd_action['action'] |
155 | 157 | ||