I got it.<br>
I will update it just as it had done in  $sht(...).<br>
<br>
On Mon, Feb 25, 2019 at 2:45 PM Daniel-Constantin Mierla <<br>
notifications@github.com> wrote:<br>
<br>
> It doesn't seem safe what you do there, because you copy the value of the<br>
> item in a new variable, then unlock the slot. But the value can be a string<br>
> and that will point in share memory. By unlocking, the item in hash table<br>
> can be deleted or modified, making the previous string value invalid.<br>
><br>
> See how is the function for pv get of $sht(...), there is a deep cloning<br>
> of the item.<br>
><br>
> —<br>
> You are receiving this because you authored the thread.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/kamailio/kamailio/pull/1866#issuecomment-466973400>,<br>
> or mute the thread<br>
> <https://github.com/notifications/unsubscribe-auth/ArE2rFNa0SskW3IB0Pc8LulNFOoGAfHrks5vQ8XfgaJpZM4bPk66><br>
> .<br>
><br>
<br>
<br>
-- <br>
--Mojtaba Esfandiari.S<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/1866#issuecomment-466980937">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZVeqNP9xRBm7wAg26PLf2T1HSxO8ks5vQ8wdgaJpZM4bPk66">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZZ6cvZWWfLPO3cQsvywQP0928pdAks5vQ8wdgaJpZM4bPk66.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@mojtabaesfandiari in #1866: I got it.\nI will update it just as it had done in  $sht(...).\n\nOn Mon, Feb 25, 2019 at 2:45 PM Daniel-Constantin Mierla \u003c\nnotifications@github.com\u003e wrote:\n\n\u003e It doesn't seem safe what you do there, because you copy the value of the\n\u003e item in a new variable, then unlock the slot. But the value can be a string\n\u003e and that will point in share memory. By unlocking, the item in hash table\n\u003e can be deleted or modified, making the previous string value invalid.\n\u003e\n\u003e See how is the function for pv get of $sht(...), there is a deep cloning\n\u003e of the item.\n\u003e\n\u003e —\n\u003e You are receiving this because you authored the thread.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/kamailio/kamailio/pull/1866#issuecomment-466973400\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/ArE2rFNa0SskW3IB0Pc8LulNFOoGAfHrks5vQ8XfgaJpZM4bPk66\u003e\n\u003e .\n\u003e\n\n\n-- \n--Mojtaba Esfandiari.S\n"}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1866#issuecomment-466980937"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/1866#issuecomment-466980937",
"url": "https://github.com/kamailio/kamailio/pull/1866#issuecomment-466980937",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>