size_t cache_len = strlen(db->handle->dbpath) + strlen(db->handle->root); Do we actually need to recalculate this each time? Maybe its worth cacheing somewhere. I'm sure there's more validation that could be done within pacman. I'll leave the min length for now.