playing with the testcheck.py yields errors

Jeffrey Blank blank at eclipse.ncsc.mil
Tue Feb 5 18:26:12 UTC 2013


That's odd.

Thanks for the suggested patch -- I just wanna do some quick checks and
make sure that the obvious fix won't breaking some kind of hidden
dependency somewhere, and then I'll respond.


On 02/05/2013 12:07 PM, Brian Millett wrote:
> Ok, so I've a rhel-6.3 system
> 
> [root at localhost checks]# uname -r
> 2.6.32-279.19.1.el6.x86_64
> [root at localhost checks]# rpm -qa | grep libxslt
> libxslt-1.1.26-2.el6_3.1.x86_64
> 
> I've been looking at the RHEL6/input/checks to see how it is done, and I
> wanted to test the checks.  So following the documentation
> https://fedorahosted.org/scap-security-guide/wiki/newoval, I ran
> 'sshd_idle_timeout_value=30 ./testcheck.py sshd_idle_timeout.xml' and
> got the following error:
> 
> [root at localhost checks]# sshd_idle_timeout_value=30 ./testcheck.py
> sshd_idle_timeout.xml
> external_variable with id : sshd_idle_timeout_value
> Evaluating with OVAL tempfile : /tmp/sshd_idle_timeoutUDK_FW.xml
> File '/tmp/sshd_idle_timeoutUDK_FW.xml' line 7: Element
> '{http://oval.mitre.org/XMLSchema/oval-definitions-5}definition',
> attribute 'id': [facet 'pattern'] The value
> 'oval:oval:scap-security-guide.testing:def:108' is not accepted by the
> pattern 'oval:[A-Za-z0-9_\-\.]+:def:[1-9][0-9]*'.
> 
> Looking into the code, I see that idtranslate.idtranslator already is
> adding the namespace to the id
> 
> str_id = "%s:%s:%s:%d" % (namespace_to_prefix(tagname), self.content_id,
> tagname_to_abbrev(tagname), i)
> 
> So the 'oval:oval:scap-security-guide.testing:def:108' being generated
> is redundant.  the following patch fixes that.
> 
> [root at localhost checks]# diff -wruN testcheck.py.orig ./testcheck.py
> --- testcheck.py.orig    2013-02-05 10:54:44.579854555 -0600
> +++ ./testcheck.py    2013-02-05 10:54:57.889884072 -0600
> @@ -102,7 +102,7 @@
>              if element.getchildren():
>                  ovaltree.append(element)
>          # re-map all the element ids from meaningful names to
> meaningless numbers
> -        testtranslator = idtranslate.idtranslator("testids.ini",
> "oval:scap-security-guide.testing")
> +        testtranslator = idtranslate.idtranslator("testids.ini",
> "scap-security-guide.testing")
>          ovaltree = testtranslator.translate(ovaltree)
>          (ovalfile, fname) = tempfile.mkstemp(prefix=defname,suffix=".xml")
>          os.write(ovalfile, ET.tostring(ovaltree))
> 
> 
> so now
> 
> [root at localhost checks]# sshd_idle_timeout_value=30 ./testcheck.py
> sshd_idle_timeout.xml
> external_variable with id : sshd_idle_timeout_value
> Evaluating with OVAL tempfile : /tmp/sshd_idle_timeoutt_n33_.xml
> Definition oval:scap-security-guide.testing:def:111: false
> Definition oval:scap-security-guide.testing:def:109: false
> Definition oval:scap-security-guide.testing:def:108: false
> Evaluation done.
> 
> -- 
> Brian Millett
>  "Shifts in paradigms
>              often cause nose bleeds."
> Greg Glenn
> 
> 
> _______________________________________________
> scap-security-guide mailing list
> scap-security-guide at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/scap-security-guide


More information about the scap-security-guide mailing list