sql query - right or wrong?

Question

Can u pls tell me is this query is right or wrong for

serach data between two dates:

from and to date is select by date pickers:


sql = "SELECT * FROM dheelabs_remedies.logs where calltime
between '" + FromDate.Text + "' and  '" + ToDate.Text + "' and remarks='" + drop_filter.SelectedItem + "' order by id desc";


Help me