diff -r a180113055cb -r 83d6a149c755 smf/smfservermodule/smfclient/smfgallery_p.cpp
--- a/smf/smfservermodule/smfclient/smfgallery_p.cpp	Thu Sep 23 17:43:31 2010 +0530
+++ b/smf/smfservermodule/smfclient/smfgallery_p.cpp	Mon Oct 11 21:59:54 2010 +0530
@@ -61,6 +61,23 @@
  */
 SmfError SmfGalleryPrivate::albums(QStringList names, SmfContact* user, int pageNum, int perPage)
 	{
+	for(int iCount = 0;iCount<names.length();iCount++)
+		{
+		if(names.isEmpty() || names[iCount].isNull() || !(names[iCount].isSimpleText())
+		   ||(isdigit(names[iCount].toInt())))
+			{
+			qDebug()<<"Invalid Names or user";	
+			return SmfInvalidDataInArguement;
+			}
+		}
+		if((pageNum<0 && perPage >0) || (pageNum>0 && perPage <0) || (pageNum ==0 && perPage ==0) 
+			|| (pageNum > 100  &&  perPage >0) || (pageNum > 0 && perPage > 30) || (isalpha(pageNum)&& isalpha(perPage)))
+    
+	
+			{
+			 qDebug()<<"Invalid Arguments, Check for Negative Values, Large Values, Junk Values";
+			 return SmfInvalidDataInArguement;
+			}
 	SmfError err = SmfNoError;
 	//We need to pass Opcode and SmfProvider serialized into bytearray 
 	SmfProvider* m_baseProvider = m_gallery->getProvider();
@@ -113,6 +130,15 @@
  */
 SmfError SmfGalleryPrivate::pictures(SmfPictureAlbumList &albums,int pageNum,int perPage)
 	{
+	if((pageNum<0 && perPage >0) || (pageNum>0 && perPage <0) || (pageNum ==0 && perPage ==0) 
+		|| (pageNum > 100  &&  perPage >0) || (pageNum > 0 && perPage > 30) || (isalpha(pageNum)&& isalpha(perPage)))
+					         
+		{
+				    
+		qDebug()<<"Invalid Arguments, Check for Negative Values, Large Values, Junk Values";
+		SmfError error = SmfInvalidDataInArguement;
+		return error;
+		}
 	SmfError err = SmfNoError;
 	//We need to pass Opcode and SmfProvider serialized into bytearray 
 	SmfProvider* m_baseProvider = m_gallery->getProvider();