Fixes on prop227 comments, based on comments by mcs on #10395

diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index dbdfff1440b91b9945dc58fb389497aeee1514a7..4a6c50616c04a7021aae68c816b3411e66a389cc 100644 (file)
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -3258,7 +3258,26 @@ connection_dirserv_flushed_some(dir_connection_t *conn)
   }
 }
 
-/** Return true iff <b>line</b> is a valid recommened_packages line.
+/** Return true iff <b>line</b> is a valid RecommenedPackages line.
+ */
+/*
+  The grammar is:
+
+    "package" SP PACKAGENAME SP VERSION SP URL SP DIGESTS NL
+
+      PACKAGENAME = NONSPACE
+      VERSION = NONSPACE
+      URL = NONSPACE
+      DIGESTS = DIGEST | DIGESTS SP DIGEST
+      DIGEST = DIGESTTYPE "=" DIGESTVAL
+
+      NONSPACE = one or more non-space printing characters
+
+      DIGESTVAL = any number of non-=, non-" " characters.
+
+      SP = " "
+      NL = a newline
+
  */
 int
 validate_recommended_package_line(const char *line)

diff --git a/src/or/or.h b/src/or/or.h
index 228738faab5a9637f3ff6e024bad1d394ed47a9b..33fc1a46b1bea3a12fb964383f1a5fe823e36d61 100644 (file)
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -2456,7 +2456,7 @@ typedef struct networkstatus_t {
   /** Vote only: what methods is this voter willing to use? */
   smartlist_t *supported_methods;
 
-  /** List of 'package' lines describing hashes of downloadable pacakges */
+  /** List of 'package' lines describing hashes of downloadable packages */
   smartlist_t *package_lines;
 
   /** How long does this vote/consensus claim that authorities take to