{
/**
- * This is a linked list.
+ * This is a doubly linked list.
*/
struct IndexInfo *next;
+ /**
+ * This is a doubly linked list.
+ */
+ struct IndexInfo *prev;
+
/**
* Name of the indexed file. Memory allocated
* at the end of this struct (do not free).
/**
- * Linked list of indexed files.
+ * Head of linked list of indexed files.
+ */
+static struct IndexInfo *indexed_files_head;
+
+/**
+ * Tail of linked list of indexed files.
*/
-static struct IndexInfo *indexed_files;
+static struct IndexInfo *indexed_files_tail;
/**
- * Maps hash over content of indexed files to the respective filename.
+ * Maps hash over content of indexed files to the respective 'struct IndexInfo'.
* The filenames are pointers into the indexed_files linked list and
* do not need to be freed.
*/
GNUNET_free (fn);
return;
}
- pos = indexed_files;
- while (pos != NULL)
- {
+ for (pos = indexed_files_head; NULL != pos; pos = pos->next)
if ((GNUNET_OK !=
GNUNET_BIO_write (wh, &pos->file_id, sizeof (struct GNUNET_HashCode))) ||
(GNUNET_OK != GNUNET_BIO_write_string (wh, pos->filename)))
break;
- pos = pos->next;
- }
if (GNUNET_OK != GNUNET_BIO_write_close (wh))
{
GNUNET_log (GNUNET_ERROR_TYPE_ERROR | GNUNET_ERROR_TYPE_BULK,
pos->filename = (const char *) &pos[1];
memcpy (&pos[1], fname, slen);
if (GNUNET_SYSERR ==
- GNUNET_CONTAINER_multihashmap_put (ifm, &hc, (void *) pos->filename,
+ GNUNET_CONTAINER_multihashmap_put (ifm, &pos->file_id, pos,
GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY))
{
GNUNET_free (pos);
}
else
{
- pos->next = indexed_files;
- indexed_files = pos;
+ GNUNET_CONTAINER_DLL_insert (indexed_files_head,
+ indexed_files_tail,
+ pos);
}
GNUNET_free (fname);
}
static void
signal_index_ok (struct IndexInfo *ii)
{
+ struct IndexInfo *ir;
if (GNUNET_SYSERR ==
GNUNET_CONTAINER_multihashmap_put (ifm, &ii->file_id,
- (void *) ii->filename,
+ ii,
GNUNET_CONTAINER_MULTIHASHMAPOPTION_UNIQUE_ONLY))
{
+ ir = GNUNET_CONTAINER_multihashmap_get (ifm,
+ &ii->file_id);
+ GNUNET_assert (NULL != ir);
GNUNET_log (GNUNET_ERROR_TYPE_INFO,
_
("Index request received for file `%s' is already indexed as `%s'. Permitting anyway.\n"),
ii->filename,
- (const char *) GNUNET_CONTAINER_multihashmap_get (ifm,
- &ii->file_id));
+ ir->filename);
GNUNET_SERVER_transmit_context_append_data (ii->tc, NULL, 0,
GNUNET_MESSAGE_TYPE_FS_INDEX_START_OK);
GNUNET_SERVER_transmit_context_run (ii->tc, GNUNET_TIME_UNIT_MINUTES);
GNUNET_free (ii);
return;
}
- ii->next = indexed_files;
- indexed_files = ii;
+ GNUNET_CONTAINER_DLL_insert (indexed_files_head,
+ indexed_files_tail,
+ ii);
write_index_list ();
GNUNET_SERVER_transmit_context_append_data (ii->tc, NULL, 0,
GNUNET_MESSAGE_TYPE_FS_INDEX_START_OK);
tc = GNUNET_SERVER_transmit_context_create (client);
iim = (struct IndexInfoMessage *) buf;
- pos = indexed_files;
- while (NULL != pos)
+ for (pos = indexed_files_head; NULL != pos; pos = pos->next)
{
fn = pos->filename;
slen = strlen (fn) + 1;
iim->file_id = pos->file_id;
memcpy (&iim[1], fn, slen);
GNUNET_SERVER_transmit_context_append_message (tc, &iim->header);
- pos = pos->next;
}
GNUNET_SERVER_transmit_context_append_data (tc, NULL, 0,
GNUNET_MESSAGE_TYPE_FS_INDEX_LIST_END);
{
const struct UnindexMessage *um;
struct IndexInfo *pos;
- struct IndexInfo *prev;
- struct IndexInfo *next;
struct GNUNET_SERVER_TransmitContext *tc;
int found;
return;
}
found = GNUNET_NO;
- prev = NULL;
- pos = indexed_files;
- while (NULL != pos)
+ for (pos = indexed_files_head; NULL != pos; pos = pos->next)
{
- next = pos->next;
if (0 == memcmp (&pos->file_id, &um->file_id, sizeof (struct GNUNET_HashCode)))
{
- if (prev == NULL)
- indexed_files = next;
- else
- prev->next = next;
+ GNUNET_CONTAINER_DLL_remove (indexed_files_head,
+ indexed_files_tail,
+ pos);
GNUNET_break (GNUNET_OK ==
GNUNET_CONTAINER_multihashmap_remove (ifm, &pos->file_id,
- (void *)
- pos->filename));
+ pos));
GNUNET_free (pos);
found = GNUNET_YES;
+ break;
}
- else
- {
- prev = pos;
- }
- pos = next;
}
GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
"Client requested unindexing of file `%s': %s\n",
const char *fn;
struct GNUNET_DISK_FileHandle *fh;
uint64_t off;
+ struct IndexInfo *ii;
if (size != sizeof (struct OnDemandBlock))
{
}
odb = (const struct OnDemandBlock *) data;
off = GNUNET_ntohll (odb->offset);
- fn = (const char *) GNUNET_CONTAINER_multihashmap_get (ifm, &odb->file_id);
+ ii = GNUNET_CONTAINER_multihashmap_get (ifm, &odb->file_id);
+ if (NULL == ii)
+ {
+ GNUNET_break (0);
+ return GNUNET_SYSERR;
+ }
+ fn = ii->filename;
if ((NULL == fn) || (0 != ACCESS (fn, R_OK)))
{
GNUNET_STATISTICS_update (GSF_stats,
{
struct IndexInfo *pos;
- GNUNET_CONTAINER_multihashmap_destroy (ifm);
- ifm = NULL;
- while (NULL != (pos = indexed_files))
+ while (NULL != (pos = indexed_files_head))
{
- indexed_files = pos->next;
+ GNUNET_CONTAINER_DLL_remove (indexed_files_head,
+ indexed_files_tail,
+ pos);
if (pos->fhc != NULL)
GNUNET_CRYPTO_hash_file_cancel (pos->fhc);
+ GNUNET_break (GNUNET_OK ==
+ GNUNET_CONTAINER_multihashmap_remove (ifm,
+ &pos->file_id, pos));
GNUNET_free (pos);
}
+ GNUNET_CONTAINER_multihashmap_destroy (ifm);
+ ifm = NULL;
cfg = NULL;
}
{
cfg = c;
dsh = d;
- ifm = GNUNET_CONTAINER_multihashmap_create (128, GNUNET_NO);
+ ifm = GNUNET_CONTAINER_multihashmap_create (128, GNUNET_YES);
read_index_list ();
return GNUNET_OK;
}