[cvsnt] cvs chown creates malformed fileattr.xml

[Mike Wake]

Hi Tony,

Does this help zoom in on the problem?

In chown.cpp:chown_dirproc() the directory owner is set like this.
    fileattr_setvalue(NULL,"directory/owner",chown_user);

Whereas in perms.cpp:change_owner() the directory owner is set like this.
    fileattr_setvalue(fileattr_create(NULL,"directory/owner"),NULL,user);

The XmlApi lib would need alot more study to figure out if the problem 
is in there or is just in the way it is used.

Cheers
Mikew.

Mike Wake wrote:
> Hi all,
> 
> Issuing
> cvs chown mwake
> using cvsnt-2.0.58d
> 
> from within a sandbox containing the CVSROOT directory results in a 
> malformed fileattr.xml.
> 
> <?xml version="1.0" encoding="UTF-8"?>
> <fileattr>
>   <directory>
>     <acl user="mwake">
>       <control />
>       <create />
>       <read />
>       <tag />
>       <write />
>     </acl>
>   </directory>
>   <directory/owner>mwake</directory/owner>
> </fileattr>
> 
> cvs lsacl results in the following output
> 
> cvs lsacl: warning: unrecognized response `Error in xml_read: not 
> well-formed (invalid token) at line 11' from cvs server
> cvs lsacl: warning: unrecognized response `Error in xml_read: not 
> well-formed (invalid token) at line 11' from cvs server
> cvs lsacl: warning: unrecognized response `Error in xml_read: not 
> well-formed (invalid token) at line 11' from cvs server
> cvs [server aborted]: Malformed fileattr.xml file in /Blah/CVSROOT/CVS. 
>  Please fix or delete this file
> 
> Cheers
> Mikew
> _______________________________________________
> cvsnt mailing list
> cvsnt at cvsnt.org
> http://www.cvsnt.org/cgi-bin/mailman/listinfo/cvsnt