And, of course, as soon as I send these, I notice the bad signoff on about half of them, including this one :( Those probably got screwed up when I ended up losing my entire repo once when a test went wrong. Anyway, I went ahead and fixed them in my local version, so next rev they'll be fine. On Wed, 2015-09-02 at 21:54 -0400, David Turner wrote: > This function might be used by other refs backends > > Signed-off-by: David Turner <dturner@xxxxxxxxxxx> > --- > refs.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/refs.h b/refs.h > index 5542587..359e166 100644 > --- a/refs.h > +++ b/refs.h > @@ -259,6 +259,17 @@ int verify_refname_available(const char *newname, struct string_list *extra, > struct string_list *skip, struct strbuf *err); > > /* > + * Check if a refname is safe. > + * For refs that start with "refs/" we consider it safe as long they do > + * not try to resolve to outside of refs/. > + * > + * For all other refs we only consider them safe iff they only contain > + * upper case characters and '_' (like "HEAD" AND "MERGE_HEAD", and not like > + * "config"). > + */ > +int refname_is_safe(const char *refname); > + > +/* > * Flags controlling ref_transaction_update(), ref_transaction_create(), etc. > * REF_NODEREF: act on the ref directly, instead of dereferencing > * symbolic references. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html