[PATCH] cleanfs: Clear /var/tmp on boot as well

[PATCH] cleanfs: Clear /var/tmp on boot as well

[Peter Müller]

[-- Attachment #1: Type: text/plain, Size: 1082 bytes --]

Similar to /tmp/, there is no reason to keep any leftovers in /var/tmp,
nor can any application expect content placed there to be persistent.

On several IPFire installations I have access to, this would remove
quite some clutter accumulated in /var/tmp over the years.

Signed-off-by: Peter Müller <peter.mueller(a)ipfire.org>
---
 src/initscripts/system/cleanfs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/initscripts/system/cleanfs b/src/initscripts/system/cleanfs
index d1cbb2547..f315682ce 100644
--- a/src/initscripts/system/cleanfs
+++ b/src/initscripts/system/cleanfs
@@ -117,6 +117,11 @@ case "${1}" in
 		find . -xdev -mindepth 1 ! -name lost+found \
 			-delete || failed=1
 
+		boot_mesg -n " /var/tmp" ${NORMAL}
+		cd /var/tmp &&
+		find . -xdev -mindepth 1 ! -name lost+found \
+			-delete || failed=1
+
 		boot_mesg -n " /var/ipfire/dhcp" ${NORMAL}
 		cd /var/ipfire/dhcpc/ && find . -name "*.pid" -exec rm -f {} \; || failed=1
 		cd /var/ipfire/dhcpc/ && find . -name "*.cache" -exec rm -f {} \; || failed=1
-- 
2.34.1

Re: [PATCH] cleanfs: Clear /var/tmp on boot as well

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 2193 bytes --]

Hello everyone,

I agree with Paul and Stefan here. We should move things out of there first before we add this directory to the tidy up at boot time.

Who would like to grab this and work on this?

-Michael

> On 28 Feb 2022, at 18:48, Stefan Schantl <stefan.schantl(a)ipfire.org> wrote:
> 
> Hello Peter, Hello List,
> 
> I have to decline this patch because the IDS system currently stores
> the downloaded tarballs (which contains the rules of a provider) in
> that directory.
> 
> Each time the ruleset is altered or updated, these tarballs will be
> decompressed and the required files are extracted from them.
> 
> So cleaning up this directory, would remove the tarballs and currently
> breaks the entire IDS.
> 
> We might have to think about keeping this tarballs or to move them to a
> different (better) location.
> 
> Best regards,
> 
> -Stefan
>> Similar to /tmp/, there is no reason to keep any leftovers in
>> /var/tmp,
>> nor can any application expect content placed there to be persistent.
>> 
>> On several IPFire installations I have access to, this would remove
>> quite some clutter accumulated in /var/tmp over the years.
>> 
>> Signed-off-by: Peter Müller <peter.mueller(a)ipfire.org>
>> ---
>>  src/initscripts/system/cleanfs | 5 +++++
>>  1 file changed, 5 insertions(+)
>> 
>> diff --git a/src/initscripts/system/cleanfs
>> b/src/initscripts/system/cleanfs
>> index d1cbb2547..f315682ce 100644
>> --- a/src/initscripts/system/cleanfs
>> +++ b/src/initscripts/system/cleanfs
>> @@ -117,6 +117,11 @@ case "${1}" in
>>                 find . -xdev -mindepth 1 ! -name lost+found \
>>                         -delete || failed=1
>>  
>> +               boot_mesg -n " /var/tmp" ${NORMAL}
>> +               cd /var/tmp &&
>> +               find . -xdev -mindepth 1 ! -name lost+found \
>> +                       -delete || failed=1
>> +
>>                 boot_mesg -n " /var/ipfire/dhcp" ${NORMAL}
>>                 cd /var/ipfire/dhcpc/ && find . -name "*.pid" -exec
>> rm -f {} \; || failed=1
>>                 cd /var/ipfire/dhcpc/ && find . -name "*.cache" -exec
>> rm -f {} \; || failed=1
> 
>