Print

Print


Hi Giuseppe,

Can you do a pull request for this change? That way you will get 
recognition for fixing it.

Andy


On Tue, 2 Nov 2021, Giuseppe Della Ricca wrote:

> Dear experts,
>
> mimicking the protection on 'char *user' used by XrdOfsFile::XrdOfsFile() in
>
> https://github.com/xrootd/xrootd/blob/fd23efca7f1b2bb5bf2d2cd70f060c1ec9385bc6/src/XrdOfs/XrdOfs.cc#L438
>
> I modified XrdThrottle::File::File()
>
> https://github.com/xrootd/xrootd/blob/0e460d7a11702c9935ed9fc1c67f22cb998b6fa4/src/XrdThrottle/XrdThrottleFile.cc#L35
>
> replacing
>
>     m_user(user),
>
> with
>
>     m_user(user ? user : ""),
>
> and this fixed the crash reported above, without visible drawbacks.
>
> @abh3 , @bbockelm , what do you think about this fix ? Thanks !
>
> Best regards,
> Giuseppe.
>
>
> -- 
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub:
> https://github.com/xrootd/xrootd/issues/1544#issuecomment-957313066


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/issues/1544#issuecomment-957921623

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1