diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-01-27 15:41:22 -0800 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-01-28 12:44:14 +0000 |
| 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 c08e2ce1f5..a143aed355 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 | ||
