projects/pwm: pwfile.c annotate

annotate pwfile.c @ 4:b5c4267a7182

Fix check for successful deletion of a empty group A return value of NULL means that there is no empty group of that name. Improve the error message.

author	Guido Berhoerster <guido+pwm@berhoerster.name>
date	Sun, 05 Feb 2017 13:30:31 +0100
parents	1cc98a5677d9
children	0b1bce8db371

rev	line source
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	1 /*
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	2 * Copyright (C) 2016 Guido Berhoerster <guido+pwm@berhoerster.name>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	3 *
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	4 * Permission is hereby granted, free of charge, to any person obtaining
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	5 * a copy of this software and associated documentation files (the
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	6 * "Software"), to deal in the Software without restriction, including
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	7 * without limitation the rights to use, copy, modify, merge, publish,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	8 * distribute, sublicense, and/or sell copies of the Software, and to
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	9 * permit persons to whom the Software is furnished to do so, subject to
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	10 * the following conditions:
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	11 *
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	12 * The above copyright notice and this permission notice shall be included
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	13 * in all copies or substantial portions of the Software.
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	14 *
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	15 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	16 * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	17 * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	18 * IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	19 * CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	20 * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	21 * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	22 */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	23
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	24 #include "compat.h"
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	25
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	26 #ifdef HAVE_ERR_H
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	27 #include <err.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	28 #endif /* HAVE_ERR_H */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	29 #include <errno.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	30 #include <stdio.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	31 #include <string.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	32 #include <sys/stat.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	33 #ifdef HAVE_SYS_TREE_H
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	34 #include <sys/tree.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	35 #endif
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	36 #include <unistd.h>
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	37
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	38 #include "pwfile.h"
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	39 #include "util.h"
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	40
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	41 struct record_id_entry {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	42 RB_ENTRY(record_id_entry) record_id_entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	43 unsigned int id;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	44 unsigned char uuid[PWS3_UUID_SIZE];
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	45 };
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	46
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	47 RB_HEAD(record_id_tree, record_id_entry);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	48
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	49 static int record_id_entry_cmp(struct record_id_entry *,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	50 struct record_id_entry *);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	51 RB_PROTOTYPE_STATIC(record_id_tree, record_id_entry, record_id_entry,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	52 record_id_entry_cmp)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	53 RB_GENERATE_STATIC(record_id_tree, record_id_entry, record_id_entry,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	54 record_id_entry_cmp)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	55
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	56 static int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	57 record_id_entry_cmp(struct record_id_entry *entry1,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	58 struct record_id_entry *entry2)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	59 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	60 if (entry1->id > entry2->id) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	61 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	62 } else if (entry1->id < entry2->id) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	63 return (1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	64 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	65 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	66 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	67
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	68 static int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	69 pws_record_cmp(const void p1, const void p2)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	70 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	71 int retval;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	72 struct pws3_record record1 = (struct pws3_record **)p1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	73 struct pws3_record record2 = (struct pws3_record **)p2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	74 struct pws3_field *group_field1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	75 const char *group1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	76 struct pws3_field *group_field2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	77 const char *group2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	78 struct pws3_field *title_field1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	79 const char *title1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	80 struct pws3_field *title_field2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	81 const char *title2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	82
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	83 group_field1 = pws3_record_get_field(record1, PWS3_RECORD_FIELD_GROUP);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	84 group1 = (group_field1 != NULL) ? pws3_field_get_text(group_field1) :
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	85 "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	86 group_field2 = pws3_record_get_field(record2, PWS3_RECORD_FIELD_GROUP);
2 5cd0debdb7d8 Fix a typo potentially causing a NULL pointer dereference Guido Berhoerster <guido+pwm@berhoerster.name> parents: 0 diff changeset	87 group2 = (group_field2 != NULL) ? pws3_field_get_text(group_field2) :
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	88 "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	89 retval = strcmp(group1, group2);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	90 if (retval != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	91 return (retval);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	92 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	93
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	94 title_field1 = pws3_record_get_field(record1, PWS3_RECORD_FIELD_TITLE);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	95 title1 = (title_field1 != NULL) ? pws3_field_get_text(title_field1) :
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	96 "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	97 title_field2 = pws3_record_get_field(record2, PWS3_RECORD_FIELD_TITLE);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	98 title2 = (title_field2 != NULL) ? pws3_field_get_text(title_field2) :
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	99 "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	100 return (strcmp(title1, title2));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	101 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	102
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	103 static void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	104 record_id_tree_clear(struct record_id_tree *tree)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	105 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	106 struct record_id_entry *entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	107 struct record_id_entry *entry_tmp;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	108
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	109 RB_FOREACH_SAFE(entry, record_id_tree, tree, entry_tmp) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	110 RB_REMOVE(record_id_tree, tree, entry);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	111 free(entry);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	112 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	113 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	114
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	115 static void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	116 record_id_tree_destroy(struct record_id_tree *tree)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	117 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	118 if (tree == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	119 return;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	120 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	121
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	122 record_id_tree_clear(tree);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	123 free(tree);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	124 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	125
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	126 static const unsigned char *
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	127 record_id_tree_get_uuid(struct record_id_tree *tree, unsigned int id)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	128 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	129 struct record_id_entry *entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	130
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	131 entry = RB_FIND(record_id_tree, tree,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	132 &(struct record_id_entry){ .id = id });
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	133 if (entry == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	134 return (NULL);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	135 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	136
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	137 return (entry->uuid);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	138 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	139
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	140 void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	141 pwfile_init(struct pwm_ctx *ctx)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	142 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	143 ctx->file = pws3_file_create();
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	144 if (ctx->file == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	145 err(1, "pws3_file_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	146 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	147
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	148 ctx->next_id = 1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	149
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	150 ctx->record_id_tree = xmalloc(sizeof (struct record_id_tree));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	151 RB_INIT(ctx->record_id_tree);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	152 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	153
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	154 void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	155 pwfile_destroy(struct pwm_ctx *ctx)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	156 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	157 record_id_tree_destroy(ctx->record_id_tree);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	158 ctx->record_id_tree = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	159 pws3_file_destroy(ctx->file);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	160 ctx->file = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	161 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	162
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	163 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	164 pwfile_read_file(struct pwm_ctx ctx, FILE fp)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	165 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	166 struct pws3_record *pws3_record;
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	167 size_t record_list_len = 0;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	168 struct pws3_record **pws3_record_list;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	169 size_t i;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	170 struct pws3_field *uuid_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	171 const unsigned char *uuid;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	172 struct record_id_entry *entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	173
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	174 if (pws3_file_read_stream(ctx->file, ctx->password, fp) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	175 warnx("failed to read password database: %s",
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	176 pws3_file_get_error_message(ctx->file));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	177 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	178 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	179
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	180 record_id_tree_clear(ctx->record_id_tree);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	181
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	182 /* sort records by group and title */
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	183 for (pws3_record = pws3_file_first_record(ctx->file);
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	184 pws3_record != NULL;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	185 pws3_record = pws3_file_next_record(ctx->file, pws3_record)) {
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	186 record_list_len++;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	187 }
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	188 pws3_record_list = xmalloc(sizeof (struct pws3_record )
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	189 record_list_len);
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	190 for (pws3_record = pws3_file_first_record(ctx->file), i = 0;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	191 pws3_record != NULL;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	192 pws3_record = pws3_file_next_record(ctx->file, pws3_record)) {
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	193 pws3_record_list[i++] = pws3_record;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	194 }
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	195 qsort(pws3_record_list, record_list_len,
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	196 sizeof (struct pws3_record *), pws_record_cmp);
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	197
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	198 /* build the tree of record IDs */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	199 for (i = 0; i < record_list_len; i++) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	200 uuid_field = pws3_record_get_field(pws3_record_list[i],
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	201 PWS3_RECORD_FIELD_UUID);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	202 uuid = pws3_field_get_uuid(uuid_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	203
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	204 entry = xmalloc(sizeof (struct record_id_entry));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	205 entry->id = ctx->next_id++;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	206 memcpy(entry->uuid, uuid, sizeof (entry->uuid));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	207
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	208 RB_INSERT(record_id_tree, ctx->record_id_tree, entry);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	209 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	210
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	211 free(pws3_record_list);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	212
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	213 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	214 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	215
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	216 static int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	217 make_backup_copy(const char *filename)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	218 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	219 int retval = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	220 FILE *fp_orig = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	221 char *backup_filename = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	222 char *tmpfilename = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	223 mode_t old_mode;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	224 int fd_backup = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	225 unsigned char buf[BUFSIZ];
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	226 size_t read_len;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	227 FILE *fp_backup = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	228
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	229 fp_orig = fopen(filename, "r");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	230 if (fp_orig == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	231 if (errno != ENOENT) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	232 warn("fopen");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	233 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	234 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	235 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	236 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	237
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	238 xasprintf(&backup_filename, "%s~", filename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	239 xasprintf(&tmpfilename, "%s.XXXXXX", filename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	240
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	241 /* create temporary file */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	242 old_mode = umask(S_IRWXG \| S_IRWXO);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	243 fd_backup = mkstemp(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	244 umask(old_mode);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	245 if (fd_backup == -1) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	246 warn("mkstemp");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	247 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	248 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	249 fp_backup = fdopen(fd_backup, "w");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	250 if (fp_backup == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	251 warn("fdopen");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	252 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	253 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	254
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	255 /* copy file contents */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	256 while (!feof(fp_orig)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	257 read_len = fread(buf, 1, sizeof (buf), fp_orig);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	258 if ((read_len < sizeof (buf)) && ferror(fp_orig)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	259 warn("fread");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	260 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	261 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	262 if (fwrite(buf, 1, read_len, fp_backup) != read_len) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	263 warn("fwrite");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	264 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	265 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	266 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	267 if (fflush(fp_backup) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	268 warn("fflush");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	269 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	270 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	271 if (fsync(fileno(fp_backup)) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	272 warn("fsync");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	273 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	274 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	275
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	276 retval = 0;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	277
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	278 out:
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	279 if ((fd_backup != -1) && (fp_backup == NULL)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	280 close(fd_backup);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	281 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	282 if (fp_backup != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	283 fclose(fp_backup);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	284 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	285 if (fp_orig != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	286 fclose(fp_orig);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	287 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	288 if (retval == 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	289 /* rename temporary file and overwrite existing file */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	290 if (rename(tmpfilename, backup_filename) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	291 warn("rename");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	292 retval = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	293 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	294 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	295 if ((retval != 0) && ((fd_backup != -1) \|\| (fp_backup != NULL))) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	296 unlink(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	297 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	298 free(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	299 free(backup_filename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	300
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	301 return (retval);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	302 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	303
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	304 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	305 pwfile_write_file(struct pwm_ctx *ctx)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	306 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	307 int retval = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	308 char *tmpfilename = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	309 mode_t old_mode;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	310 int fd = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	311 FILE *fp = NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	312
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	313 if (make_backup_copy(ctx->filename) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	314 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	315 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	316
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	317 xasprintf(&tmpfilename, "%s.XXXXXX", ctx->filename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	318
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	319 /* create temporary file */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	320 old_mode = umask(S_IRWXG \| S_IRWXO);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	321 fd = mkstemp(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	322 if (fd == -1) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	323 warn("mkstemp");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	324 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	325 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	326 umask(old_mode);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	327 fp = fdopen(fd, "w");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	328 if (fp == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	329 warn("fdopen");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	330 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	331 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	332
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	333 /* write contents */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	334 if (pws3_file_write_stream(ctx->file, ctx->password, 10000, fp) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	335 warnx("pws3_file_write_stream: %s",
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	336 pws3_file_get_error_message(ctx->file));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	337 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	338 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	339 if (fflush(fp) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	340 warn("fflush");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	341 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	342 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	343 if (fsync(fileno(fp)) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	344 warn("fsync");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	345 goto out;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	346 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	347
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	348 retval = 0;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	349
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	350 out:
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	351 if ((fd != -1) && (fp == NULL)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	352 close(fd);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	353 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	354 if (fp != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	355 fclose(fp);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	356 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	357 if (retval == 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	358 /* rename temporary file and overwrite existing file */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	359 if (rename(tmpfilename, ctx->filename) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	360 warn("rename");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	361 retval = -1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	362 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	363 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	364 if ((retval != 0) && ((fd != -1) \|\| (fp != NULL))) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	365 unlink(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	366 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	367 free(tmpfilename);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	368
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	369 return (retval);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	370 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	371
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	372 static int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	373 list_item_cmp(const void p1, const void p2)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	374 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	375 int retval;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	376 const union list_item item1 = (const union list_item **)p1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	377 const union list_item item2 = (const union list_item **)p2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	378 const char *group1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	379 const char *group2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	380 const char *title1;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	381 const char *title2;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	382
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	383 /* sort both groups and records first by group name */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	384 group1 = (item1->any.group != NULL) ? item1->any.group : "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	385 group2 = (item2->any.group != NULL) ? item2->any.group : "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	386 retval = strcmp(group1, group2);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	387 if ((retval != 0) \|\| ((item1->any.type == ITEM_TYPE_GROUP) &&
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	388 (item2->any.type == ITEM_TYPE_GROUP))) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	389 return (retval);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	390 } else if ((item1->any.type == ITEM_TYPE_GROUP) &&
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	391 (item2->any.type == ITEM_TYPE_RECORD)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	392 /* groups come before records belonging to it */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	393 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	394 } else if ((item1->any.type == ITEM_TYPE_RECORD) &&
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	395 (item2->any.type == ITEM_TYPE_GROUP)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	396 return (1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	397 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	398
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	399 /* sort records also by title */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	400 title1 = (item1->record.title != NULL) ? item1->record.title : "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	401 title2 = (item2->record.title != NULL) ? item2->record.title : "";
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	402 return (strcmp(title1, title2));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	403 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	404
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	405 union list_item **
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	406 pwfile_create_list(struct pwm_ctx *ctx)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	407 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	408 union list_item **list;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	409 struct record_id_entry *entry;
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	410 size_t list_capacity = 0;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	411 struct pws3_field *empty_group_field;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	412 struct pws3_record *pws3_record;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	413 struct pws3_field *group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	414 const char *group;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	415 struct pws3_field *title_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	416 const char *title;
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	417 union list_item *item;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	418 size_t list_len = 0;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	419 size_t i;
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	420 size_t j;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	421 const char *prev_group = NULL;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	422
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	423 RB_FOREACH(entry, record_id_tree, ctx->record_id_tree) {
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	424 list_capacity++;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	425 }
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	426 list_capacity = 2; / maximum number of group items */
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	427 for (empty_group_field = pws3_file_first_empty_group(ctx->file);
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	428 empty_group_field != NULL;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	429 empty_group_field = pws3_file_next_empty_group(ctx->file,
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	430 empty_group_field)) {
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	431 list_capacity++;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	432 }
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	433 list_capacity++; /* terminating NULL */
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	434 list = xmalloc(sizeof (union list_item ) list_capacity);
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	435
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	436 RB_FOREACH(entry, record_id_tree, ctx->record_id_tree) {
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	437 pws3_record = pws3_file_get_record(ctx->file, entry->uuid);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	438 group_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	439 PWS3_RECORD_FIELD_GROUP);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	440 group = (group_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	441 pws3_field_get_text(group_field) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	442 title_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	443 PWS3_RECORD_FIELD_TITLE);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	444 title = (title_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	445 pws3_field_get_text(title_field) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	446
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	447 item = xmalloc(sizeof (union list_item));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	448 item->record.type = ITEM_TYPE_RECORD;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	449 item->record.group = (group != NULL) ? xstrdup(group) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	450 item->record.title = (title != NULL) ? xstrdup(title) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	451 item->record.id = entry->id;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	452 memcpy(item->record.uuid, entry->uuid,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	453 sizeof (item->record.uuid));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	454
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	455 list[list_len++] = item;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	456 }
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	457 /* sort records by group and title in order to find unqiue groups */
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	458 qsort(list, list_len, sizeof (union list_item *), list_item_cmp);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	459
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	460 /* add groups based on the sorted records */
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	461 for (i = 0, j = list_len; i < list_len; i++) {
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	462 group = list[i]->record.group;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	463 if ((group != NULL) && ((prev_group == NULL) \|\|
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	464 (strcmp(prev_group, group) != 0))) {
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	465 item = xmalloc(sizeof (union list_item));
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	466 item->group.type = ITEM_TYPE_GROUP;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	467 item->group.group = (group != NULL) ? xstrdup(group) :
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	468 NULL;
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	469 list[j++] = item;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	470 prev_group = group;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	471 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	472 }
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	473 list_len = j;
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	474
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	475 /* add empty groups to the list */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	476 for (empty_group_field = pws3_file_first_empty_group(ctx->file);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	477 empty_group_field != NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	478 empty_group_field = pws3_file_next_empty_group(ctx->file,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	479 empty_group_field)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	480 group = pws3_field_get_text(empty_group_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	481
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	482 item = xmalloc(sizeof (union list_item));
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	483 item->group.type = ITEM_TYPE_GROUP;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	484 item->group.group = xstrdup(group);
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	485
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	486 list[list_len++] = item;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	487 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	488
3 1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	489 /* terminate the list */
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	490 list[list_len] = NULL;
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	491 list = xrealloc(list, sizeof (union list_item ) (list_len + 1));
1cc98a5677d9 Refactor the creation of the record and group list Guido Berhoerster <guido+pwm@berhoerster.name> parents: 2 diff changeset	492
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	493 /* sort the final list by group and title */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	494 qsort(list, list_len, sizeof (union list_item *), list_item_cmp);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	495
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	496 return (list);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	497 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	498
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	499 void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	500 pwfile_destroy_list(union list_item **list)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	501 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	502 size_t i;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	503
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	504 if (list == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	505 return;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	506 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	507
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	508 for (i = 0; list[i] != NULL; i++) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	509 if (list[i]->any.type == ITEM_TYPE_RECORD) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	510 free(list[i]->record.title);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	511 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	512 free(list[i]->any.group);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	513 free(list[i]);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	514 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	515
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	516 free(list);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	517 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	518
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	519 static void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	520 update_record(struct pws3_record pws3_record, struct record record)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	521 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	522 struct pws3_field *title_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	523 struct pws3_field *group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	524 struct pws3_field *username_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	525 struct pws3_field *password_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	526 struct pws3_field *notes_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	527 struct pws3_field *url_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	528
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	529 if (record->title != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	530 title_field = pws3_field_create(0, PWS3_RECORD_FIELD_TITLE);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	531 if (title_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	532 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	533 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	534 if (pws3_field_set_text(title_field,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	535 record->title) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	536 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	537 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	538 pws3_record_set_field(pws3_record, title_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	539 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	540 if (record->group != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	541 group_field = pws3_field_create(0, PWS3_RECORD_FIELD_GROUP);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	542 if (group_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	543 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	544 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	545 if (pws3_field_set_text(group_field,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	546 record->group) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	547 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	548 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	549 pws3_record_set_field(pws3_record, group_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	550 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	551 if (record->username != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	552 username_field = pws3_field_create(0,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	553 PWS3_RECORD_FIELD_USERNAME);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	554 if (username_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	555 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	556 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	557 if (pws3_field_set_text(username_field,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	558 record->username) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	559 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	560 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	561 pws3_record_set_field(pws3_record, username_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	562 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	563 if (record->password != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	564 password_field = pws3_field_create(0,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	565 PWS3_RECORD_FIELD_PASSWORD);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	566 if (password_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	567 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	568 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	569 if (pws3_field_set_text(password_field,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	570 record->password) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	571 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	572 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	573 pws3_record_set_field(pws3_record, password_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	574 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	575 if (record->notes != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	576 notes_field = pws3_field_create(0, PWS3_RECORD_FIELD_NOTES);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	577 if (notes_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	578 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	579 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	580 if (pws3_field_set_text(notes_field, record->notes) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	581 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	582 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	583 pws3_record_set_field(pws3_record, notes_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	584 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	585 if (record->url != NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	586 url_field = pws3_field_create(0, PWS3_RECORD_FIELD_URL);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	587 if (url_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	588 err(1, "pws3_record_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	589 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	590 if (pws3_field_set_text(url_field, record->url) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	591 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	592 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	593 pws3_record_set_field(pws3_record, url_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	594 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	595 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	596
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	597 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	598 pwfile_create_record(struct pwm_ctx ctx, struct record record)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	599 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	600 struct pws3_record *pws3_record;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	601 const unsigned char *uuid;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	602 struct record_id_entry *entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	603
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	604 pws3_record = pws3_record_create();
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	605 if (pws3_record == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	606 err(1, "pws3_record_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	607 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	608 update_record(pws3_record, record);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	609 pws3_file_insert_record(ctx->file, pws3_record);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	610
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	611 uuid = pws3_field_get_uuid(pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	612 PWS3_RECORD_FIELD_UUID));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	613 entry = xmalloc(sizeof (struct record_id_entry));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	614 entry->id = ctx->next_id++;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	615 memcpy(entry->uuid, uuid, sizeof (entry->uuid));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	616 RB_INSERT(record_id_tree, ctx->record_id_tree, entry);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	617
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	618 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	619 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	620
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	621 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	622 pwfile_modify_record(struct pwm_ctx *ctx, unsigned int id,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	623 struct record *record)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	624 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	625 const unsigned char *uuid;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	626
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	627 uuid = record_id_tree_get_uuid(ctx->record_id_tree, id);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	628 if (uuid == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	629 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	630 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	631
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	632 update_record(pws3_file_get_record(ctx->file, uuid), record);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	633
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	634 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	635 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	636
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	637 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	638 pwfile_remove_record(struct pwm_ctx *ctx, unsigned int id)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	639 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	640 const unsigned char *uuid;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	641 struct record_id_entry *entry;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	642
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	643 uuid = record_id_tree_get_uuid(ctx->record_id_tree, id);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	644 if (uuid == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	645 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	646 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	647
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	648 pws3_record_destroy(pws3_file_remove_record(ctx->file, uuid));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	649
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	650 entry = RB_FIND(record_id_tree, ctx->record_id_tree,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	651 &(struct record_id_entry){ .id = id });
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	652 free(RB_REMOVE(record_id_tree, ctx->record_id_tree, entry));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	653
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	654 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	655 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	656
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	657 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	658 pwfile_create_group(struct pwm_ctx ctx, const char group)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	659 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	660 struct pws3_record *pws3_record;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	661 struct pws3_field *group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	662 struct pws3_field *empty_group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	663
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	664 /* check for a record in the given group */
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	665 for (pws3_record = pws3_file_first_record(ctx->file);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	666 pws3_record != NULL; pws3_record = pws3_file_next_record(ctx->file,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	667 pws3_record)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	668 group_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	669 PWS3_RECORD_FIELD_GROUP);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	670 if ((group_field != NULL) &&
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	671 (strcmp(group, pws3_field_get_text(group_field)) == 0)) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	672 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	673 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	674 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	675
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	676 empty_group_field = pws3_field_create(1,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	677 PWS3_HEADER_FIELD_EMPTY_GROUPS);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	678 if (empty_group_field == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	679 err(1, "pws3_field_create");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	680 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	681 if (pws3_field_set_text(empty_group_field, group) != 0) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	682 err(1, "pws3_field_set_text");
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	683 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	684 pws3_file_insert_empty_group(ctx->file, empty_group_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	685
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	686 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	687 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	688
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	689 int
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	690 pwfile_remove_group(struct pwm_ctx ctx, const char group)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	691 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	692 struct pws3_field *empty_group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	693
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	694 empty_group_field = pws3_file_remove_empty_group(ctx->file, group);
4 b5c4267a7182 Fix check for successful deletion of a empty group Guido Berhoerster <guido+pwm@berhoerster.name> parents: 3 diff changeset	695 if (empty_group_field == NULL) {
0 a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	696 return (-1);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	697 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	698 pws3_field_destroy(empty_group_field);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	699
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	700 return (0);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	701 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	702
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	703 struct record *
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	704 pwfile_get_record(struct pwm_ctx *ctx, unsigned int id)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	705 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	706 struct record *record;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	707 const unsigned char *uuid;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	708 struct pws3_record *pws3_record;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	709 struct pws3_field *title_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	710 struct pws3_field *group_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	711 struct pws3_field *username_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	712 struct pws3_field *password_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	713 struct pws3_field *notes_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	714 struct pws3_field *url_field;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	715
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	716 uuid = record_id_tree_get_uuid(ctx->record_id_tree, id);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	717 if (uuid == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	718 return (NULL);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	719 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	720 pws3_record = pws3_file_get_record(ctx->file, uuid);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	721
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	722 record = xmalloc(sizeof (struct record));
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	723
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	724 title_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	725 PWS3_RECORD_FIELD_TITLE);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	726 record->title = (title_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	727 xstrdup(pws3_field_get_text(title_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	728
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	729 group_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	730 PWS3_RECORD_FIELD_GROUP);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	731 record->group = (group_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	732 xstrdup(pws3_field_get_text(group_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	733
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	734 username_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	735 PWS3_RECORD_FIELD_USERNAME);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	736 record->username = (username_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	737 xstrdup(pws3_field_get_text(username_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	738
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	739 password_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	740 PWS3_RECORD_FIELD_PASSWORD);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	741 record->password = (password_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	742 xstrdup(pws3_field_get_text(password_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	743
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	744 notes_field = pws3_record_get_field(pws3_record,
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	745 PWS3_RECORD_FIELD_NOTES);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	746 record->notes = (notes_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	747 xstrdup(pws3_field_get_text(notes_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	748
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	749 url_field = pws3_record_get_field(pws3_record, PWS3_RECORD_FIELD_URL);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	750 record->url = (url_field != NULL) ?
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	751 xstrdup(pws3_field_get_text(url_field)) : NULL;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	752
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	753 return (record);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	754 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	755
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	756 void
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	757 pwfile_destroy_record(struct record *record)
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	758 {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	759 if (record == NULL) {
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	760 return;
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	761 }
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	762
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	763 free(record->title);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	764 free(record->group);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	765 free(record->username);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	766 free(record->password);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	767 free(record->notes);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	768 free(record->url);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	769 free(record);
a7e41e1a79c8 Initial revision Guido Berhoerster <guido+pwm@berhoerster.name> parents: diff changeset	770 }

Mercurial > projects > pwm

annotate pwfile.c @ 4:b5c4267a7182