Merge pull request #1108 from ffurano/fix-cr-lf

Prevent the logic that looks for the /n from getting confused by line…
xrootd · Jan 9, 2020 · 347524c · 347524c
2 parents fc9f9f2 + e6db780
commit 347524c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/XrdHttp/XrdHttpProtocol.cc b/src/XrdHttp/XrdHttpProtocol.cc
@@ -678,7 +678,7 @@ int XrdHttpProtocol::Process(XrdLink *lp) // We ignore the argument here
     while ((rc = BuffgetLine(tmpline)) > 0) {
       TRACE(DEBUG, " rc:" << rc << " got hdr line: " << tmpline);
 
-      if ((rc == 2) && (tmpline[rc - 1] == '\n')) {
+      if ((rc == 2) && (tmpline.length() > 1) && (tmpline[rc - 1] == '\n')) {
         CurrentReq.headerok = true;
         TRACE(DEBUG, " rc:" << rc << " detected header end.");
         break;