Opened 17 years ago

Closed 16 years ago

Last modified 15 years ago

#78 closed Defect (fixed)

Broken delete_file [patch]

Reported by: Nicolas Owned by: Bruce Allen
Priority: Minor Milestone: Undetermined
Component: Server - Scheduler Version:
Keywords: delete_file patch Cc:

Description

delete_file is storing incorrect data on the msg_to_host database table. Variety and xml fields are empty. This is because the mth structure is cleared after these fields are set, on function delete_host_file. Patch attached.

Additionally, there is extra code to check if -host_id is 'all':

if(!strcmp(argv[++i], "all")) {
    host_id = 0;
}

but some lines below the program exits if host_id == 0. Also, the scheduler has no special code to handle this. The messages will never be sent because host '0' will never communicate. I'm not sure how this would be fixed, since I don't know what was the intended behavior.

Attachments (1)

delete_file.diff (683 bytes) - added by Nicolas 17 years ago.
Patch

Download all attachments as: .zip

Change History (4)

Changed 17 years ago by Nicolas

Attachment: delete_file.diff added

Patch

comment:1 Changed 17 years ago by Nicolas

Summary: Broken delete_fileBroken delete_file [patch]

Add patch availability to ticket title.

comment:2 Changed 17 years ago by Nicolas

Keywords: patch added

comment:3 Changed 16 years ago by davea

Resolution: fixed
Status: newclosed

(In [14434]) - API: check for messages from client 10X a second;

this will allow smoother CPU throttling

  • delete_file: bug fixes (from Nicolas Alvarez). Fixes #78
Note: See TracTickets for help on using tickets.