Stefano Brivio <sbrivio@xxxxxxxxxx> writes:
> /* BB eventually switch this to SMB2 specific small buf size */
> - *request_buf = cifs_small_buf_get();
> + if (smb2_command == SMB2_SET_INFO)
> + *request_buf = cifs_buf_get();
> + else
> + *request_buf = cifs_small_buf_get();
> if (*request_buf == NULL) {
> /* BB should we add a retry in here if not a writepage? */
> return -ENOMEM;
> @@ -3720,7 +3723,7 @@ send_set_info(const unsigned int xid, struct cifs_tcon *tcon,
>
> rc = cifs_send_recv(xid, ses, &rqst, &resp_buftype, flags,
> &rsp_iov);
> - cifs_small_buf_release(req);
> + cifs_buf_release(req);
> rsp = (struct smb2_set_info_rsp *)rsp_iov.iov_base;
Small and large bufs use different mempools, shouldn't the release func
match the get func?
--
Aurélien Aptel / SUSE Labs Samba Team
GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3
SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
