features/changelog: NULL termination after memcpy should be of dest pointer

snippet code
<
 ..memcpy (bufff, src, len - 1);
 ..*(src + len) = '\0'; ---> Wrong!
>

Source buffer lvalue() referencing with offset style NULL
termination is wrong and unnecessary when we have a destination
buffer, it is the destination buffer which should look to be NULL
terminated

Makes it more readable and also clearly logical.

<
 ..memcpy (bufff, src, len - 1);
 ..bufff[len -1] = '\0'; ---> Correct!
>

Change-Id: I6d7f312aaa5c541f0345649ff1ef9f193892b674
BUG: 1099986
Signed-off-by: Harshavardhana <harsha@harshavardhana.net>
Reviewed-on: http://review.gluster.org/7836
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Tested-by: Venky Shankar <vshankar@redhat.com>