Bug 105914 - hald exit if you add a non existing callout to a fdi file
Summary: hald exit if you add a non existing callout to a fdi file
Status: RESOLVED FIXED
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: Basesystem (show other bugs)
Version: Beta 2
Hardware: Other All
: P5 - None : Critical
Target Milestone: ---
Assignee: Kay Sievers
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-19 19:05 UTC by Danny Al-Gaaf
Modified: 2005-08-26 07:38 UTC (History)
0 users

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
strace from hal for daemon=yes (4.05 KB, text/plain)
2005-08-19 19:08 UTC, Danny Al-Gaaf 		Details
strace from HAL with correct callout (4.87 KB, application/octet-stream)
2005-08-19 19:09 UTC, Danny Al-Gaaf 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Danny Al-Gaaf 2005-08-19 19:05:55 UTC 
I changed the fdi file for hald-subfs-mount (90-bock-subfs.fdi), but you can 
also reproduce this by change 90-fstab-sync.fdi e.g. on fedora, and added a non 
existing callout (changed hal-subfs-mount to hal-subfs-mountt).

As I restarted hal as daemon (and only in this case) hal dies without any 
errormessage. After a strace I notice HAL didn't die, HAL exited (and cleaned 
up). This should not be the correct behavior. I can't find the reason for this. 

I attach a part of strace with and without changed fdi file for the related 
callout.
Comment 1 Danny Al-Gaaf 2005-08-19 19:08:06 UTC 
Created attachment 46722 [details]
strace from hal for daemon=yes

This part of strace was from HAL in daemon mode with the changed, non existing
callout. If you start HAL with --daemon=no in strace nothing happens. No exit.
Comment 2 Danny Al-Gaaf 2005-08-19 19:09:59 UTC 
Created attachment 46723 [details]
strace from HAL with correct callout
Comment 3 Kay Sievers 2005-08-26 01:56:30 UTC 
Hopefully fixed. Patch is on the hal mailinglist. Please try...
Comment 4 Danny Al-Gaaf 2005-08-26 07:38:14 UTC 
tested, works fine. Added to new package. ;)