Re: Undeletable files after kyua test runs

> On 29. Jun 2020, at 21:18, Ian Lepore <ian_at_freebsd.org> wrote:
> 
> On Mon, 2020-06-29 at 21:08 +0200, Gordon Bergling wrote:
>> On Mon, Jun 29, 2020 at 11:58:47AM -0700, Rodney W. Grimes wrote:
>>>> On Mon, Jun 29, 2020 at 10:32:38AM -0700, Kevin Oberman wrote:
>>>>> On Mon, Jun 29, 2020 at 10:26 AM Gordon Bergling <
>>>>> gbe_at_freebsd.org> wrote:
>>>>>> I recently stumbled across undeletable files that are
>>>>>> generated by kyua
>>>>>> test runs,
>>>>>> for example
>>>>>> 
>>>>>> -rwxr-xr-x  1 root  wheel  0 May  9 13:10
>>>>>> /tmp/kyua.aB4q62/8676/work/fileforaudit
>>>>>> 
>>>>>> I haven't yet identified the test that generate those files,
>>>>>> but it is
>>>>>> impossible
>>>>>> to delete them. I have clear_tmp_enable="YES" set in the
>>>>>> /etc/rc.conf, but
>>>>>> on every boot the system argues that these file aren't
>>>>>> deletable.
>>>>>> I tried to 'rm -rf' them by hand but, even this wasn't
>>>>>> possible. I have
>>>>>> looked for
>>>>>> any extend attributes, but I didn't find any.
>>>>>> 
>>>>>> Has anyone an idea how this is possible and may how these
>>>>>> files can be
>>>>>> deleted?
>>>>> 
>>>>> Have you done 'ls -o' to check for flags like schg?
>>>>> --
>>>>> Kevin Oberman, Part time kid herder and retired Network
>>>>> Engineer
>>>>> E-mail: rkoberman_at_gmail.com
>>>>> PGP Fingerprint: D03FB98AFA78E3B78C1694B318AB39EF1B055683
>>>> 
>>>> Argh, I haven't thought about chflags for quite some time. The
>>>> chflags
>>>> bit was set and after an
>>>> 
>>>> # find /tmp/ -type f -exec chflags -R 0 {} \;
>>> 
>>>               ^^Only files          ^^ meaningless when chflags is
>>> given ONLY files
>>> 
>>> You probably could of done:
>>> chflags -R 0 /tmp/
>> 
>> Okay, I am currently working on an update for clear_tmp_enable="YES"
>> to include
>> a check like this. I would think that an rc option like this should
>> delete 
>> everything in /tmp.
>> 
> 
> I disagree.  One of the few things those immutable flags are good for
> is protecting files from things like an rc script or other automation
> that deletes files.  Those flags are typically set and maintained by
> users and admins, and automation should not change them in order to
> delete files.
> 
> The real fix we need is for the kyua tests to properly clean up after
> themselves, including fixing the flags on temporary files created or
> used by the tests, and then deleting them.
> 

+1, having a routine script remove schg automatically IMHO defeats the purpose of setting this flag.

Cheers,
Michael