diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-12-14 12:27:33 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-12-14 12:36:04 +0000 |
| commit | dc637ae860f57161c97b04d3e91612d3e1600f37 (patch) | |
| tree | ddb8a1e23817d7aba466b395421142b26357bd6d /bitbake/lib/bb/command.py | |
| parent | 1572c1517c96d6556a832b9511957d311a5f457d (diff) | |
| download | poky-dc637ae860f57161c97b04d3e91612d3e1600f37.tar.gz | |
bitbake: command: Fix getCmdLineAction bugs
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>
Diffstat (limited to 'bitbake/lib/bb/command.py')
| -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 39a2e18a02..c842497e63 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 | ||
