diff -r ead42e28e519 -r 79bd6bd83ec3 securitydialogs/SecUi/Src/SecUiManualSecuritySettings.cpp
--- a/securitydialogs/SecUi/Src/SecUiManualSecuritySettings.cpp	Wed Sep 15 13:09:05 2010 +0300
+++ b/securitydialogs/SecUi/Src/SecUiManualSecuritySettings.cpp	Wed Oct 13 15:30:47 2010 +0300
@@ -399,40 +399,44 @@
     RDebug::Print(_L("(SECUI)CManualSecuritySettings::LockSimL(): Param length: %d"), length);
     #endif
 
-    if (aType.CompareF(Operator) == 0)
-        {
-        aLockType = RMmCustomAPI::EOperator;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
-        }
-
-    if (aType.CompareF(Gid1) == 0)
-        {
-        aLockType = RMmCustomAPI::EGid1;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
-        }
-
-    if (aType.CompareF(Gid2) == 0)
+    // make sure aLockCode's length won't exceed KSimLockMaxPasswordSize
+    if( aLockCode.Length() <= KSimLockMaxPasswordSize )
         {
-        aLockType = RMmCustomAPI::EGid2;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
-        }
+        if ( aType.CompareF( Operator ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EOperator;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
 
-    if (aType.CompareF(Imsi) == 0)
-        {
-        aLockType = RMmCustomAPI::EImsi;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
-        }
+        if ( aType.CompareF( Gid1 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EGid1;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
+
+        if ( aType.CompareF( Gid2 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EGid2;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
 
-    if (aType.CompareF(Operator_Gid1) == 0)
-        {    
-        aLockType = RMmCustomAPI::EOperator_Gid1;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
-        }
+        if ( aType.CompareF( Imsi ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EImsi;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
 
-    if (aType.CompareF(Operator_Gid2) == 0)
-        {
-        aLockType = RMmCustomAPI::EOperator_Gid2;
-        ret = iCustomPhone.ActivateSimLock( aLockCode,aLockType );
+        if ( aType.CompareF( Operator_Gid1 ) == 0 )
+            {    
+            aLockType = RMmCustomAPI::EOperator_Gid1;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
+
+        if ( aType.CompareF( Operator_Gid2 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EOperator_Gid2;
+            ret = iCustomPhone.ActivateSimLock( aLockCode, aLockType );
+            }
         }
 
     #if defined(_DEBUG)
@@ -500,48 +504,51 @@
     #if defined(_DEBUG)
     RDebug::Print(_L("(SECUI)CManualSecuritySettings::UnLockSimL(): Param length: %d"), length);
     #endif
-    
-    if (aType.CompareF(Operator) == 0)
-        {
-        aLockType = RMmCustomAPI::EOperator;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }
 
-    if (aType.CompareF(Gid1) == 0)
-        {
-        aLockType = RMmCustomAPI::EGid1;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }
-
-    if (aType.CompareF(Gid2) == 0)
-        {
-        aLockType = RMmCustomAPI::EGid2;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }
-
-    if (aType.CompareF(Imsi) == 0)
+    // make sure aUnlockCode's length won't exceed KSimLockMaxPasswordSize
+    if( aUnlockCode.Length() <= KSimLockMaxPasswordSize )
         {
-        aLockType = RMmCustomAPI::EImsi;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }
+        if ( aType.CompareF( Operator ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EOperator;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
 
-    if (aType.CompareF(Operator_Gid1) == 0)
-        {    
-        aLockType = RMmCustomAPI::EOperator_Gid1;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }
+        if ( aType.CompareF( Gid1 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EGid1;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
+
+        if ( aType.CompareF( Gid2 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EGid2;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
 
-    if (aType.CompareF(Operator_Gid2) == 0)
-        {
-        aLockType = RMmCustomAPI::EOperator_Gid2;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
+        if ( aType.CompareF( Imsi ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EImsi;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
+
+        if ( aType.CompareF( Operator_Gid1 ) == 0 )
+            {    
+            aLockType = RMmCustomAPI::EOperator_Gid1;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
+
+        if ( aType.CompareF( Operator_Gid2 ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EOperator_Gid2;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
+        if ( aType.CompareF( GlobalUnlock ) == 0 )
+            {
+            aLockType = RMmCustomAPI::EGlobalUnlock;
+            ret = iCustomPhone.DeActivateSimLock( aUnlockCode, aLockType );
+            }
         }
-    if (aType.CompareF(GlobalUnlock) == 0)
-        {
-        aLockType = RMmCustomAPI::EGlobalUnlock;
-        ret = iCustomPhone.DeActivateSimLock( aUnlockCode,aLockType );
-        }    
-        
     #if defined(_DEBUG)
     RDebug::Print(_L("(SECUI)CManualSecuritySettings::UnLockSimL() RESULT: %d"), ret);
     #endif