12 files changed, 61 insertions, 70 deletions

diff --git a/linden/indra/llmessage/llassetstorage.cpp b/linden/indra/llmessage/llassetstorage.cpp
index 0ab1081..fd1dd95 100644
--- a/linden/indra/llmessage/llassetstorage.cpp
+++ b/linden/indra/llmessage/llassetstorage.cpp
@@ -38,7 +38,7 @@
 
 #include "llassetstorage.h"
 
-// linden library includes
+// viewer library includes
 #include "llmath.h"
 #include "llstring.h"
 #include "lldir.h"
diff --git a/linden/indra/llmessage/llcachename.cpp b/linden/indra/llmessage/llcachename.cpp
index 88a0cfe..db81e8c 100644
--- a/linden/indra/llmessage/llcachename.cpp
+++ b/linden/indra/llmessage/llcachename.cpp
@@ -34,7 +34,7 @@
 
 #include "llcachename.h"
 
-// linden library includes
+// viewer library includes
 #include "lldbstrings.h"
 #include "llframetimer.h"
 #include "llhost.h"
diff --git a/linden/indra/llmessage/lldatapacker.cpp b/linden/indra/llmessage/lldatapacker.cpp
index b746b5a..8270ddf 100644
--- a/linden/indra/llmessage/lldatapacker.cpp
+++ b/linden/indra/llmessage/lldatapacker.cpp
@@ -671,7 +671,7 @@ void LLDataPackerBinaryBuffer::dumpBufferToLog()
         S32 cur_line = 0;
         for (i = 0; i < mBufferSize; i++)
         {
-                snprintf(line_buffer + cur_line_pos*3, sizeof(line_buffer) - cur_line_pos*3, "%02x ", mBufferp[i]);     /* Flawfinder: ignore */
+                ll_snprintf(line_buffer + cur_line_pos*3, sizeof(line_buffer) - cur_line_pos*3, "%02x ", mBufferp[i]);  /* Flawfinder: ignore */
                 cur_line_pos++;
                 if (cur_line_pos >= 16)
                 {
@@ -696,14 +696,14 @@ BOOL LLDataPackerAsciiBuffer::packString(const std::string& value, const char *n
         int numCopied = 0;
         if (mWriteEnabled) 
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\n", value.c_str());               /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\n", value.c_str());            /* Flawfinder: ignore */
         }
         else
         {
                 numCopied = value.length() + 1; /*Flawfinder: ignore*/
         }
 
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -741,9 +741,9 @@ BOOL LLDataPackerAsciiBuffer::packBinaryData(const U8 *value, S32 size, const ch
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%010d ", size);      /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%010d ", size);   /* Flawfinder: ignore */
 
-                // snprintf returns number of bytes that would have been
+                // ll_snprintf returns number of bytes that would have been
                 // written had the output not being truncated. In that case,
                 // it will retuen >= passed in size value.  so a check needs
                 // to be added to detect truncation, and if there is any, only
@@ -761,7 +761,7 @@ BOOL LLDataPackerAsciiBuffer::packBinaryData(const U8 *value, S32 size, const ch
                 BOOL bBufferFull = FALSE;
                 for (i = 0; i < size && !bBufferFull; i++)
                 {
-                        numCopied = snprintf(mCurBufferp, getBufferSize()-getCurrentSize(), "%02x ", value[i]); /* Flawfinder: ignore */
+                        numCopied = ll_snprintf(mCurBufferp, getBufferSize()-getCurrentSize(), "%02x ", value[i]);      /* Flawfinder: ignore */
                         if (numCopied < 0 || numCopied > getBufferSize()-getCurrentSize())
                         {
                                 numCopied = getBufferSize()-getCurrentSize();
@@ -773,7 +773,7 @@ BOOL LLDataPackerAsciiBuffer::packBinaryData(const U8 *value, S32 size, const ch
 
                 if (!bBufferFull)
                 {
-                        numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(), "\n");       /* Flawfinder: ignore */
+                        numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(), "\n");    /* Flawfinder: ignore */
                         if (numCopied < 0 || numCopied > getBufferSize()-getCurrentSize())
                         {
                                 numCopied = getBufferSize()-getCurrentSize();
@@ -835,7 +835,7 @@ BOOL LLDataPackerAsciiBuffer::packBinaryDataFixed(const U8 *value, S32 size, con
                 BOOL bBufferFull = FALSE;
                 for (i = 0; i < size && !bBufferFull; i++)
                 {
-                        numCopied = snprintf(mCurBufferp, getBufferSize()-getCurrentSize(), "%02x ", value[i]); /* Flawfinder: ignore */
+                        numCopied = ll_snprintf(mCurBufferp, getBufferSize()-getCurrentSize(), "%02x ", value[i]);      /* Flawfinder: ignore */
                         if (numCopied < 0 || numCopied > getBufferSize()-getCurrentSize())
                         {
                             numCopied = getBufferSize()-getCurrentSize();
@@ -847,7 +847,7 @@ BOOL LLDataPackerAsciiBuffer::packBinaryDataFixed(const U8 *value, S32 size, con
                 }
                 if (!bBufferFull)
                 {
-                        numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(), "\n");       /* Flawfinder: ignore */
+                        numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(), "\n");    /* Flawfinder: ignore */
                         if (numCopied < 0 || numCopied > getBufferSize()-getCurrentSize())
                         {
                                 numCopied = getBufferSize()-getCurrentSize();
@@ -901,15 +901,15 @@ BOOL LLDataPackerAsciiBuffer::packU8(const U8 value, const char *name)
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);       /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);    /* Flawfinder: ignore */
         }
         else
         {
                 // just do the write to a temp buffer to get the length
-                numCopied = snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);        /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);     /* Flawfinder: ignore */
         }
 
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -949,14 +949,14 @@ BOOL LLDataPackerAsciiBuffer::packU16(const U16 value, const char *name)
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);       /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);    /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);        /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);     /* Flawfinder: ignore */
         }
 
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -997,13 +997,13 @@ BOOL LLDataPackerAsciiBuffer::packU32(const U32 value, const char *name)
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%u\n", value);       /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%u\n", value);    /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%u\n", value);        /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%u\n", value);     /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1041,13 +1041,13 @@ BOOL LLDataPackerAsciiBuffer::packS32(const S32 value, const char *name)
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);       /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d\n", value);    /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);                /* Flawfinder: ignore */        
+                numCopied = ll_snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%d\n", value);             /* Flawfinder: ignore */        
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1085,13 +1085,13 @@ BOOL LLDataPackerAsciiBuffer::packF32(const F32 value, const char *name)
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f\n", value);               /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f\n", value);            /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%f\n", value);                /* Flawfinder: ignore */        
+                numCopied = ll_snprintf(DUMMY_BUFFER, sizeof(DUMMY_BUFFER), "%f\n", value);             /* Flawfinder: ignore */        
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1129,13 +1129,13 @@ BOOL LLDataPackerAsciiBuffer::packColor4(const LLColor4 &value, const char *name
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);      /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);    /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1172,13 +1172,13 @@ BOOL LLDataPackerAsciiBuffer::packColor4U(const LLColor4U &value, const char *na
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d %d %d %d\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%d %d %d %d\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);      /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%d %d %d %d\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);    /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%d %d %d %d\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1222,13 +1222,13 @@ BOOL LLDataPackerAsciiBuffer::packVector2(const LLVector2 &value, const char *na
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f\n", value.mV[0], value.mV[1]); /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f\n", value.mV[0], value.mV[1]);      /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f\n", value.mV[0], value.mV[1]);            /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f\n", value.mV[0], value.mV[1]);         /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1266,13 +1266,13 @@ BOOL LLDataPackerAsciiBuffer::packVector3(const LLVector3 &value, const char *na
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f\n", value.mV[0], value.mV[1], value.mV[2]); /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f\n", value.mV[0], value.mV[1], value.mV[2]);      /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f\n", value.mV[0], value.mV[1], value.mV[2]);    /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f\n", value.mV[0], value.mV[1], value.mV[2]); /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1309,13 +1309,13 @@ BOOL LLDataPackerAsciiBuffer::packVector4(const LLVector4 &value, const char *na
         int numCopied = 0;
         if (mWriteEnabled)
         {
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);      /* Flawfinder: ignore */
         }
         else
         {
-                numCopied = snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]);    /* Flawfinder: ignore */
+                numCopied = ll_snprintf(DUMMY_BUFFER,sizeof(DUMMY_BUFFER),"%f %f %f %f\n", value.mV[0], value.mV[1], value.mV[2], value.mV[3]); /* Flawfinder: ignore */
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1356,13 +1356,13 @@ BOOL LLDataPackerAsciiBuffer::packUUID(const LLUUID &value, const char *name)
         {
                 std::string tmp_str;
                 value.toString(tmp_str);
-                numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\n", tmp_str.c_str());     /* Flawfinder: ignore */
+                numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\n", tmp_str.c_str());  /* Flawfinder: ignore */
         }
         else
         {
                 numCopied = 64 + 1; // UUID + newline
         }
-        // snprintf returns number of bytes that would have been written
+        // ll_snprintf returns number of bytes that would have been written
         // had the output not being truncated. In that case, it will
         // return either -1 or value >= passed in size value . So a check needs to be added
         // to detect truncation, and if there is any, only account for the
@@ -1407,14 +1407,14 @@ void LLDataPackerAsciiBuffer::writeIndentedName(const char *name)
                 int numCopied = 0;
                 if (mWriteEnabled)
                 {
-                        numCopied = snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\t", name);        /* Flawfinder: ignore */
+                        numCopied = ll_snprintf(mCurBufferp,getBufferSize()-getCurrentSize(),"%s\t", name);     /* Flawfinder: ignore */
                 }
                 else
                 {
                         numCopied = (S32)strlen(name) + 1;      /* Flawfinder: ignore */ //name + tab   
                 }
 
-                // snprintf returns number of bytes that would have been written
+                // ll_snprintf returns number of bytes that would have been written
                 // had the output not being truncated. In that case, it will
                 // return either -1 or value >= passed in size value . So a check needs to be added
                 // to detect truncation, and if there is any, only account for the
@@ -1480,7 +1480,7 @@ std::string convertF32ToString(F32 val)
 {
         std::string str;
         char  buf[20];
-        snprintf(buf, 20, "%f", val);
+        ll_snprintf(buf, 20, "%f", val);
         str = buf;
         return str;
 }
@@ -1535,13 +1535,13 @@ BOOL LLDataPackerAsciiFile::packBinaryData(const U8 *value, S32 size, const char
         else if (mOutputStream)
         {
                 char buffer[32];        /* Flawfinder: ignore */
-                snprintf(buffer,sizeof(buffer), "%010d ", size);        /* Flawfinder: ignore */
+                ll_snprintf(buffer,sizeof(buffer), "%010d ", size);     /* Flawfinder: ignore */
                 *mOutputStream << buffer;
 
                 S32 i;
                 for (i = 0; i < size; i++)
                 {
-                        snprintf(buffer, sizeof(buffer), "%02x ", value[i]);    /* Flawfinder: ignore */
+                        ll_snprintf(buffer, sizeof(buffer), "%02x ", value[i]); /* Flawfinder: ignore */
                         *mOutputStream << buffer;
                 }
                 *mOutputStream << "\n";
@@ -1595,7 +1595,7 @@ BOOL LLDataPackerAsciiFile::packBinaryDataFixed(const U8 *value, S32 size, const
                 S32 i;
                 for (i = 0; i < size; i++)
                 {
-                        snprintf(buffer, sizeof(buffer), "%02x ", value[i]);    /* Flawfinder: ignore */
+                        ll_snprintf(buffer, sizeof(buffer), "%02x ", value[i]); /* Flawfinder: ignore */
                         *mOutputStream << buffer;
                 }
                 *mOutputStream << "\n";
diff --git a/linden/indra/llmessage/llinstantmessage.cpp b/linden/indra/llmessage/llinstantmessage.cpp
index 44be3ae..63586a5 100644
--- a/linden/indra/llmessage/llinstantmessage.cpp
+++ b/linden/indra/llmessage/llinstantmessage.cpp
@@ -60,6 +60,7 @@ const S32 VOTE_UNANIMOUS = 2;
 const char EMPTY_BINARY_BUCKET[] = "";
 const S32 EMPTY_BINARY_BUCKET_SIZE = 1;
 const U32 NO_TIMESTAMP = 0;
+// Damn, this seems to be hard coded into the protocol.
 const std::string SYSTEM_FROM("Second Life");
 const S32 IM_TTL = 1;
 
@@ -233,8 +234,8 @@ void pack_instant_message_block(
         if(!message.empty())
         {
                 char buffer[MTUBYTES];
-                int num_written = snprintf(buffer, MTUBYTES, "%s", message.c_str());    /* Flawfinder: ignore */
-                // snprintf returns number of bytes that would have been written
+                int num_written = ll_snprintf(buffer, MTUBYTES, "%s", message.c_str()); /* Flawfinder: ignore */
+                // ll_snprintf returns number of bytes that would have been written
                 // had the output not being truncated. In that case, it will
                 // return either -1 or value >= passed in size value . So a check needs to be added
                 // to detect truncation, and if there is any, only account for the
diff --git a/linden/indra/llmessage/lliosocket.cpp b/linden/indra/llmessage/lliosocket.cpp
index 686c037..150b771 100644
--- a/linden/indra/llmessage/lliosocket.cpp
+++ b/linden/indra/llmessage/lliosocket.cpp
@@ -597,7 +597,7 @@ LLIOPipe::EStatus LLIOServerSocket::process_impl(
         else
         {
                 char buf[256];
-                llwarns << "Unable to accept linden socket: " << apr_strerror(status, buf, sizeof(buf)) << llendl;
+                llwarns << "Unable to accept server socket: " << apr_strerror(status, buf, sizeof(buf)) << llendl;
         }
 
         PUMP_DEBUG;
diff --git a/linden/indra/llmessage/llloginflags.h b/linden/indra/llmessage/llloginflags.h
index 6f119e7..ceade87 100644
--- a/linden/indra/llmessage/llloginflags.h
+++ b/linden/indra/llmessage/llloginflags.h
@@ -33,7 +33,7 @@
 #ifndef LL_LLLOGINFLAGS_H
 #define LL_LLLOGINFLAGS_H
 
-// Is this your first login to Second Life?
+// Is this your first login to the grid?
 const U32       LOGIN_FLAG_FIRST_LOGIN                  = (1 << 0);
 
 // Is this a trial account?
diff --git a/linden/indra/llmessage/llpacketring.cpp b/linden/indra/llmessage/llpacketring.cpp
index a39b435..e7dad79 100644
--- a/linden/indra/llmessage/llpacketring.cpp
+++ b/linden/indra/llmessage/llpacketring.cpp
@@ -34,7 +34,7 @@
 
 #include "llpacketring.h"
 
-// linden library includes
+// viewer library includes
 #include "llerror.h"
 #include "lltimer.h"
 #include "timing.h"
diff --git a/linden/indra/llmessage/llregionflags.h b/linden/indra/llmessage/llregionflags.h
index 2324785..70c3499 100644
--- a/linden/indra/llmessage/llregionflags.h
+++ b/linden/indra/llmessage/llregionflags.h
@@ -137,15 +137,6 @@ inline U32 unset_prelude_flags(U32 flags)
                         & ~(REGION_FLAGS_PRELUDE_SET | REGION_FLAGS_SUN_FIXED));
 }
 
-// estate constants. Need to match first few etries in indra.estate table.
-const U32 ESTATE_ALL = 0; // will not match in db, reserved key for logic
-const U32 ESTATE_MAINLAND = 1;
-const U32 ESTATE_ORIENTATION = 2;
-const U32 ESTATE_INTERNAL = 3;
-const U32 ESTATE_SHOWCASE = 4;
-const U32 ESTATE_TEEN = 5;
-const U32 ESTATE_LAST_LINDEN = 5; // last linden owned/managed estate
-
 // for EstateOwnerRequest, setaccess message
 const U32 ESTATE_ACCESS_ALLOWED_AGENTS  = 1 << 0;
 const U32 ESTATE_ACCESS_ALLOWED_GROUPS  = 1 << 1;
diff --git a/linden/indra/llmessage/lltaskname.h b/linden/indra/llmessage/lltaskname.h
index 401d355..305b31a 100644
--- a/linden/indra/llmessage/lltaskname.h
+++ b/linden/indra/llmessage/lltaskname.h
@@ -44,7 +44,7 @@
 typedef enum e_lltask_name
 {
         LLTASK_NULL                             = 0, // Not a valid task
-        LLTASK_AGENT                    = 1, // The player's agent in Linden World
+        LLTASK_AGENT                    = 1, // The player's agent in world
         LLTASK_CHILD_AGENT              = 2, // Child agents sent to adjacent regions
 //      LLTASK_BASIC_SHOT,              // Simple shot that moves in a straight line
 //      LLTASK_BIG_SHOT,                // Big shot that uses gravity
diff --git a/linden/indra/llmessage/message.cpp b/linden/indra/llmessage/message.cpp
index 81f7c2a..9aaa034 100644
--- a/linden/indra/llmessage/message.cpp
+++ b/linden/indra/llmessage/message.cpp
@@ -51,7 +51,7 @@
 #include "apr_network_io.h"
 #include "apr_poll.h"
 
-// linden library headers
+// viewer library headers
 #include "indra_constants.h"
 #include "lldarray.h"
 #include "lldir.h"
@@ -3125,7 +3125,7 @@ bool LLMessageSystem::generateDigestForNumberAndUUIDs(
 
         d.update((const unsigned char *) colon, (U32)strlen(colon));    /* Flawfinder: ignore */ 
         
-        snprintf(tbuf, sizeof(tbuf),"%i", number);              /* Flawfinder: ignore */
+        ll_snprintf(tbuf, sizeof(tbuf),"%i", number);           /* Flawfinder: ignore */
         d.update((unsigned char *) tbuf, (U32)strlen(tbuf));    /* Flawfinder: ignore */ 
         
         d.update((const unsigned char *) colon, (U32)strlen(colon));    /* Flawfinder: ignore */ 
@@ -3385,7 +3385,7 @@ void LLMessageSystem::dumpPacketToLog()
         for (i = 0; i < mTrueReceiveSize; i++)
         {
                 S32 offset = cur_line_pos * 3;
-                snprintf(line_buffer + offset, sizeof(line_buffer) - offset,
+                ll_snprintf(line_buffer + offset, sizeof(line_buffer) - offset,
                                  "%02x ", mTrueReceiveBuffer.buffer[i]);        /* Flawfinder: ignore */
                 cur_line_pos++;
                 if (cur_line_pos >= 16)
diff --git a/linden/indra/llmessage/message.h b/linden/indra/llmessage/message.h
index 43c3b0a..1ef15d2 100644
--- a/linden/indra/llmessage/message.h
+++ b/linden/indra/llmessage/message.h
@@ -305,8 +305,7 @@ public:
         S32 getErrorCode() const { return mErrorCode; }
 
         // Read file and build message templates filename must point to a
-        // valid string which specifies the path of a valid linden
-        // template.
+        // valid string which specifies the path of a valid template.
         void loadTemplateFile(const std::string& filename, bool failure_is_fatal);
 
 
diff --git a/linden/indra/llmessage/net.cpp b/linden/indra/llmessage/net.cpp
index 6db7214..fa51645 100644
--- a/linden/indra/llmessage/net.cpp
+++ b/linden/indra/llmessage/net.cpp
@@ -50,7 +50,7 @@
         #include <errno.h>
 #endif
 
-// linden library includes
+// viewer library includes
 #include "llerror.h"
 #include "llhost.h"
 #include "lltimer.h"