Skip to content

Commit

Permalink
Fix issue #523.
Browse files Browse the repository at this point in the history
When argparse was added to chipsec_main it no longer requires -a arguements to be separated with a comma.
Adding fix into chipsec main to handle cases when comma separated values are given
@todo with different request:
Update any test docs to remove the comma within arguements for the modules
  • Loading branch information
BrentHoltsclaw authored and ErikBjorge committed Mar 1, 2019
1 parent d0cfbd6 commit a1c8cbe
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion chipsec_main.py
@@ -1,6 +1,6 @@
#!/usr/bin/env python
#CHIPSEC: Platform Security Assessment Framework
#Copyright (c) 2010-2018, Intel Corporation
#Copyright (c) 2010-2019, Intel Corporation
#
#This program is free software; you can redistribute it and/or
#modify it under the terms of the GNU General Public License
Expand Down Expand Up @@ -399,6 +399,9 @@ def parse_args(self):
logger().DEBUG = True
if self.log:
logger().set_log_file( self.log )
if self._module_argv and len(self._module_argv) == 1 and len(self._module_argv[0].split(',')) > 1:
logger().log( "[*] Use of the -a command no longer needs to have arguments concatinated with ','")
self._module_argv = self._module_argv[0].split(',')
if self._unknownPlatform is False:
logger().log( "[*] Ignoring unsupported platform warning and continue execution" )

Expand Down

0 comments on commit a1c8cbe

Please sign in to comment.