Browse Source

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

Nick Mathewson 9 years ago
parent
commit
1e61b45251
2 changed files with 21 additions and 2 deletions
  1. 20 1
      src/or/dirserv.c
  2. 1 1
      src/or/or.h

+ 20 - 1
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)

+ 1 - 1
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