[PR PATCH] [Closed]: lib/param: handle (ignore) substitution variable in smb.conf

[github at samba.org]

There's a closed pull request on the Samba Samba Github repository

lib/param: handle (ignore) substitution variable in smb.conf
https://github.com/samba-team/samba/pull/12
Description: Signed-off-by: Quentin Gibeaux <qgibeaux at iris-tech.fr>

Pull request of fix in regards to https://bugzilla.samba.org/show_bug.cgi?id=10722

Summary  (see bugtrack for details) : 
I've encountered an annoying bug with samba-tool, some exceptions aren't caught causing the tool to crash if it finds something it doesn't like in the smb.conf.
I have a specific line in my conf : "include = /etc/samba/%U.smb.conf ", it's useful for applying specific samba configuration for a specific user, it works on the samba side, but when samba-tool reads the conf, it crashs :

root at server:~/samba# samba-tool user list
Processing section "[global]"
(…)
Can't find include file /etc/samba/%U.smb.conf
params.c:pm_process() - Failed. Error returned from params.c:parse().
pm_process() returned No
ERROR(runtime): uncaught exception - Unable to load default file
File "/usr/lib/python2.7/dist-packages/samba/netcmd/__init__.py", line 175, in _run
return self.run(*args, **kwargs)
File "/usr/lib/python2.7/dist-packages/samba/netcmd/user.py", line 261, in run
lp = sambaopts.get_loadparm()
File "/usr/lib/python2.7/dist-packages/samba/getopt.py", line 92, in get_loadparm
self._lp.load_default()

In fact it crashes for any include with any variable substitution (%U, %g, %a…).