View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002229 | GNUnet | file-sharing service | public | 2012-03-20 01:10 | 2012-06-02 17:15 |
| Reporter | LRN | Assigned To | Christian Grothoff | Due Date | |
| Priority | high | Severity | crash | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | Git master | ||||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0002229: fip_signal_start() relies on metadata to detect directories | ||||
| Description | fip_signal_start() calls GNUNET_FS_meta_data_test_for_directory() on the metadata to detect that it's dealing with a directory and recourse into it. However, GNUnet-FS-GTK allows the user to remove metadata - including the directory mime type, which prevents fip_signal_start() from working correctly, and leads to eventual segfault. I propose to not to rely on GNUNET_FS_meta_data_test_for_directory() here, and use some other way to detect directories. | ||||
| Tags | No tags attached. | ||||
| Well, we used to take the meta data only as "advice" and if it was absent would simply 'try' if the file was a directory anyway later. Also, for directories, the filename is 'supposed' to end with "/". Now, the real issue here clearly is the crash -- nothing can stop the publisher from not putting the "/" or putting the wrong mime type (and I think it is fine if the receiver then doesn't treat the file as a directory). Can you provide a more detailed crash report? | |
| I could not reproduce the crash. However, I removed the call to test for directory-status to fs meta data in SVN 21167, so the underlying cause (if diagnosed correctly) should definitively be gone now. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-03-20 01:10 | LRN | New Issue | |
| 2012-03-20 08:36 | Christian Grothoff | Note Added: 0005645 | |
| 2012-03-20 08:37 | Christian Grothoff | Priority | low => high |
| 2012-03-20 08:37 | Christian Grothoff | Severity | tweak => crash |
| 2012-03-20 08:37 | Christian Grothoff | Product Version | => Git master |
| 2012-03-20 08:37 | Christian Grothoff | Target Version | => 0.9.3 |
| 2012-03-20 08:37 | Christian Grothoff | Status | new => feedback |
| 2012-04-26 11:35 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2012-04-26 11:35 | Christian Grothoff | Status | feedback => assigned |
| 2012-04-26 11:36 | Christian Grothoff | Note Added: 0005745 | |
| 2012-04-26 11:36 | Christian Grothoff | Status | assigned => resolved |
| 2012-04-26 11:36 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2012-04-26 11:36 | Christian Grothoff | Resolution | open => fixed |
| 2012-06-02 17:15 | Christian Grothoff | Status | resolved => closed |