[libvirt] [PATCH 04/10] Secret manipulation step 7: Local driver

Miloslav Trmac mitr at redhat.com
Tue Sep 8 12:26:20 UTC 2009


----- "Daniel Veillard" <veillard at redhat.com> wrote:
> On Mon, Sep 07, 2009 at 04:12:39PM +0200, Miloslav Trmač wrote:
> > +    if ((size_t)st.st_size != st.st_size) {
> 
>   shouldn't we chaeck against SECRET_MAX_XML_FILE instead ?
No, this code reads the secret value, not the XML, and there's little reason to impose an arbitrary limit on the size.  SECRET_MAX_XML_FILE is a left-over from an earlier version, the attached updated patch removes the definition.
    Mirek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Secret-manipulation-step-7-Local-driver.patch
Type: application/mbox
Size: 34629 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20090908/7a1f2d55/attachment-0001.mbox>


More information about the libvir-list mailing list